milestone 23: correct three comments the s1 review found inaccurate

web/src/ui/styles.test.tsx

@@ -4,9 +4,10 @@ import { styles } from "./styles";
 
 /**
  * The StyleX compile-time transform must run in the vitest pipeline, not just
- * in `vite build`. `runtimeInjection` is off, so an untransformed
- * `stylex.props` call yields no class name at all — this probe fails loudly if
- * the plugin ever drops out of the test config (milestone 23, ruling 1).
+ * in `vite build`. Without the plugin these tests do not fail an assertion —
+ * importing the module throws `Unexpected 'stylex.defineVars' call at runtime`
+ * before a single case runs, which is the louder failure of the two
+ * (milestone 23, ruling 1).
  */
 describe("the StyleX build integration", () => {
 	it("compiles stylex.props into a class name", () => {

web/src/ui/styles.ts

@@ -172,8 +172,11 @@ export const styles = stylex.create({
 		overflowX: "auto",
 	},
 	/**
-	 * `space-y-3` became a flex column with a gap: same rendered rhythm, and it
-	 * does not depend on sibling margin collapsing.
+	 * Not a faithful port: `space-y-3` set sibling margins on a block container,
+	 * this is a flex formatting context. StyleX has no way to write the `> * + *`
+	 * selector that `space-y` compiles to, so the rhythm can only come from
+	 * `gap`. Child sizing and margin behaviour differ from the Tailwind original;
+	 * both call sites are plain vertical form stacks, where they do not.
 	 */
 	formCard: {
 		display: "flex",