Apply suggestions from code review

Co-authored-by: Robin Malfait <[email protected]>

crates/node/src/lib.rs

@@ -85,10 +85,10 @@ pub struct Scanner {
 #[derive(Debug, Clone)]
 #[napi(object)]
 pub struct CandidateWithPosition {
-  // The candidate string
+  /// The candidate string
   pub candidate: String,
 
-  // The position of the candidate inside the content file
+  /// The position of the candidate inside the content file
   pub position: i64,
 }
 

packages/@tailwindcss-upgrade/src/template/candidates.ts

@@ -1,6 +1,6 @@
 import { Scanner } from '@tailwindcss/oxide'
 import stringByteSlice from 'string-byte-slice'
-import { type Candidate, type Variant } from '../../../tailwindcss/src/candidate'
+import type { Candidate, Variant } from '../../../tailwindcss/src/candidate'
 import type { DesignSystem } from '../../../tailwindcss/src/design-system'
 
 export async function extractCandidates(

packages/@tailwindcss-upgrade/src/template/codemods/migrate-important.ts

@@ -13,7 +13,7 @@ import type { Candidate } from '../../../../tailwindcss/src/candidate'
 //
 //   flex! md:block!
 export function migrateImportant(candidate: Candidate): Candidate | null {
-  if (candidate.important) {
+  if (candidate.important && candidate.raw[candidate.raw.length - 1] !== '!') {
     // The printCandidate function will already put the exclamation mark in the
     // right place, so we just need to mark this candidate as requiring a
     // migration.