Use TODO(codemod) for sibling broadening comment, tighten PR description guidance

claude · claude · commit e8149a49ddc5 · 2026-03-14T23:31:10.000Z
Change the inline comment from `// NOTE:` to `// TODO(codemod):` so it's actionable and attributable. Also tighten CLAUDE.md PR description guidance. https://claude.ai/code/session_01NACR4DTcJGrJ3xu7sWWQC9
diff --git a/CLAUDE.md b/CLAUDE.md
@@ -218,7 +218,7 @@ Skills are located in `.claude/skills/`.
 
 ## Pull Request Descriptions
 
-Keep PR descriptions concise: a brief summary of what the PR does and why. Do not add sections like "Test case", "Test plan", or checklists. Context that's obvious from the diff or commit message doesn't need repeating.
+Keep PR descriptions **very short** — 1-3 sentences max. Just say what changed and why. Do not add sections like "Summary", "Test case", "Test plan", "Changes", or checklists. Do not enumerate files changed or repeat what's obvious from the diff. Less is more.
 
 ## Post-Implementation Workflow
 
diff --git a/src/__tests__/transform.test.ts b/src/__tests__/transform.test.ts
@@ -695,7 +695,7 @@ export const App = () => <Thing />;
     expect(infoWarnings).toHaveLength(0);
     // The output should contain a NOTE comment about the broadening
     expect(result.code).toContain(
-      "// NOTE: CSS `+` (adjacent sibling) becomes `~` (general sibling) in StyleX",
+      "// TODO(codemod): CSS `+` (adjacent) was broadened to `~` (general sibling). Verify siblings are always adjacent.",
     );
   });
 
diff --git a/src/internal/lower-rules/process-rules.ts b/src/internal/lower-rules/process-rules.ts
@@ -768,7 +768,7 @@ export function processDeclRules(ctx: DeclProcessingState): void {
             value: siblingValue,
             ...(isAdjacentCombinator && {
               leadingComment:
-                "NOTE: CSS `+` (adjacent sibling) becomes `~` (general sibling) in StyleX",
+                "TODO(codemod): CSS `+` (adjacent) was broadened to `~` (general sibling). Verify siblings are always adjacent.",
             }),
           });
         }
@@ -2174,7 +2174,8 @@ function handleSiblingSelector(
       keyExpr: makeSiblingKeyExpr(),
       value: siblingValue,
       ...(isAdjacent && {
-        leadingComment: "NOTE: CSS `+` (adjacent sibling) becomes `~` (general sibling) in StyleX",
+        leadingComment:
+          "TODO(codemod): CSS `+` (adjacent) was broadened to `~` (general sibling). Verify siblings are always adjacent.",
       }),
     });
   }
diff --git a/test-cases/selector-componentSiblingCombinator.output.tsx b/test-cases/selector-componentSiblingCombinator.output.tsx
@@ -37,12 +37,12 @@ const styles = stylex.create({
     paddingInline: 8,
     color: {
       default: "gray",
-      // NOTE: CSS `+` (adjacent sibling) becomes `~` (general sibling) in StyleX
+      // TODO(codemod): CSS `+` (adjacent) was broadened to `~` (general sibling). Verify siblings are always adjacent.
       [stylex.when.siblingBefore(":focus-visible", LinkMarker)]: "blue",
     },
     backgroundColor: {
       default: null,
-      // NOTE: CSS `+` (adjacent sibling) becomes `~` (general sibling) in StyleX
+      // TODO(codemod): CSS `+` (adjacent) was broadened to `~` (general sibling). Verify siblings are always adjacent.
       [stylex.when.siblingBefore(":hover", LinkMarker)]: {
         default: null,
         "@media (min-width: 768px)": "lightyellow",
diff --git a/test-cases/selector-sibling.output.tsx b/test-cases/selector-sibling.output.tsx
@@ -37,28 +37,28 @@ const styles = stylex.create({
   thing: {
     color: {
       default: "blue",
-      // NOTE: CSS `+` (adjacent sibling) becomes `~` (general sibling) in StyleX
+      // TODO(codemod): CSS `+` (adjacent) was broadened to `~` (general sibling). Verify siblings are always adjacent.
       [stylex.when.siblingBefore(":is(*)", ThingMarker)]: "red",
     },
     paddingBlock: 8,
     paddingInline: 16,
     backgroundColor: {
       default: null,
-      // NOTE: CSS `+` (adjacent sibling) becomes `~` (general sibling) in StyleX
+      // TODO(codemod): CSS `+` (adjacent) was broadened to `~` (general sibling). Verify siblings are always adjacent.
       [stylex.when.siblingBefore(":is(*)", ThingMarker)]: "lime",
     },
   },
   thingThemed: {
     color: {
       default: "blue",
-      // NOTE: CSS `+` (adjacent sibling) becomes `~` (general sibling) in StyleX
+      // TODO(codemod): CSS `+` (adjacent) was broadened to `~` (general sibling). Verify siblings are always adjacent.
       [stylex.when.siblingBefore(":is(*)", ThingThemedMarker)]: $colors.labelBase,
     },
   },
   row: {
     marginTop: {
       default: null,
-      // NOTE: CSS `+` (adjacent sibling) becomes `~` (general sibling) in StyleX
+      // TODO(codemod): CSS `+` (adjacent) was broadened to `~` (general sibling). Verify siblings are always adjacent.
       [stylex.when.siblingBefore(":is(*)", RowMarker)]: 16,
     },
   },
diff --git a/test-cases/selector-siblingBaseAfter.output.tsx b/test-cases/selector-siblingBaseAfter.output.tsx
@@ -19,7 +19,7 @@ const styles = stylex.create({
   thing: {
     color: {
       default: "blue",
-      // NOTE: CSS `+` (adjacent sibling) becomes `~` (general sibling) in StyleX
+      // TODO(codemod): CSS `+` (adjacent) was broadened to `~` (general sibling). Verify siblings are always adjacent.
       [stylex.when.siblingBefore(":is(*)", ThingMarker)]: "red",
     },
     paddingBlock: 8,
diff --git a/test-cases/selector-siblingMarkerScoping.output.tsx b/test-cases/selector-siblingMarkerScoping.output.tsx
@@ -28,17 +28,17 @@ const styles = stylex.create({
     padding: 8,
     borderTopWidth: {
       default: null,
-      // NOTE: CSS `+` (adjacent sibling) becomes `~` (general sibling) in StyleX
+      // TODO(codemod): CSS `+` (adjacent) was broadened to `~` (general sibling). Verify siblings are always adjacent.
       [stylex.when.siblingBefore(":is(*)", RowMarker)]: 1,
     },
     borderTopStyle: {
       default: null,
-      // NOTE: CSS `+` (adjacent sibling) becomes `~` (general sibling) in StyleX
+      // TODO(codemod): CSS `+` (adjacent) was broadened to `~` (general sibling). Verify siblings are always adjacent.
       [stylex.when.siblingBefore(":is(*)", RowMarker)]: "solid",
     },
     borderTopColor: {
       default: null,
-      // NOTE: CSS `+` (adjacent sibling) becomes `~` (general sibling) in StyleX
+      // TODO(codemod): CSS `+` (adjacent) was broadened to `~` (general sibling). Verify siblings are always adjacent.
       [stylex.when.siblingBefore(":is(*)", RowMarker)]: "gray",
     },
   },
diff --git a/test-cases/selector-siblingMedia.output.tsx b/test-cases/selector-siblingMedia.output.tsx
@@ -19,7 +19,7 @@ const styles = stylex.create({
     padding: 8,
     marginTop: {
       default: null,
-      // NOTE: CSS `+` (adjacent sibling) becomes `~` (general sibling) in StyleX
+      // TODO(codemod): CSS `+` (adjacent) was broadened to `~` (general sibling). Verify siblings are always adjacent.
       [stylex.when.siblingBefore(":is(*)", ThingMarker)]: {
         default: null,
         "@media (min-width: 768px)": 16,
