@tsrx/mcp 0.0.4 → 0.0.6

package/package.json

@@ -3,7 +3,7 @@
   "description": "MCP server for TSRX documentation and project context",
   "license": "MIT",
   "author": "Dominic Gannaway",
-  "version": "0.0.4",
+  "version": "0.0.6",
   "type": "module",
   "repository": {
     "type": "git",
@@ -37,8 +37,8 @@
     "@modelcontextprotocol/sdk": "^1.29.0",
     "prettier": "^3.8.3",
     "zod": "^4.3.6",
-    "@tsrx/core": "0.0.23",
-    "@tsrx/prettier-plugin": "0.3.43"
+    "@tsrx/prettier-plugin": "0.3.45",
+    "@tsrx/core": "0.0.25"
   },
   "devDependencies": {
     "@types/node": "^24.3.0",

package/src/analyze.js

@@ -1,4 +1,12 @@
-import { DIAGNOSTIC_CODES } from '@tsrx/core';
+import {
+	COMPONENT_DO_WHILE_STATEMENT_ERROR,
+	COMPONENT_FOR_IN_STATEMENT_ERROR,
+	COMPONENT_FOR_STATEMENT_ERROR,
+	COMPONENT_LOOP_BREAK_ERROR,
+	COMPONENT_LOOP_RETURN_ERROR,
+	COMPONENT_WHILE_STATEMENT_ERROR,
+	DIAGNOSTIC_CODES,
+} from '@tsrx/core';
 import { compile_tsrx } from './compile.js';
 
 /**
@@ -28,6 +36,7 @@ function create_advice(input) {
 	/** @type {TSRXAdvice[]} */
 	const advice = [];
 	const error_codes = new Set(compileResult.errors.map((error) => error.code).filter(Boolean));
+	const error_messages = new Set(compileResult.errors.map((error) => error.message));
 
 	if (!compileResult.target) {
 		advice.push({
@@ -98,6 +107,36 @@ function create_advice(input) {
 		});
 	}
 
+	if (
+		error_messages.has(COMPONENT_LOOP_RETURN_ERROR) ||
+		error_messages.has(COMPONENT_LOOP_BREAK_ERROR)
+	) {
+		advice.push({
+			kind: 'component-loop-control-flow',
+			severity: 'error',
+			title: 'Use continue inside component for...of loops',
+			message:
+				'Top-level return and break statements are not valid inside a component for...of loop. Use continue to skip the current rendered item. Nested functions inside the loop keep ordinary JavaScript control flow.',
+			documentation: ['tsrx://docs/control-flow.md'],
+		});
+	}
+
+	if (
+		error_messages.has(COMPONENT_FOR_STATEMENT_ERROR) ||
+		error_messages.has(COMPONENT_FOR_IN_STATEMENT_ERROR) ||
+		error_messages.has(COMPONENT_WHILE_STATEMENT_ERROR) ||
+		error_messages.has(COMPONENT_DO_WHILE_STATEMENT_ERROR)
+	) {
+		advice.push({
+			kind: 'unsupported-component-loop',
+			severity: 'error',
+			title: 'Use for...of for component list rendering',
+			message:
+				'Component template scope supports for...of loops for rendering lists. Move regular for, for...in, while, and do...while loops into a nested function, event handler, effect, or helper.',
+			documentation: ['tsrx://docs/control-flow.md'],
+		});
+	}
+
 	if (advice.length === 0 && compileResult.ok) {
 		advice.push({
 			kind: 'compile-clean',

package/src/generated/docs.js

@@ -43,7 +43,7 @@ export const documentation_sections = [
 		use_cases:
 			'if else, for loops, switch, try catch, conditional rendering, lists, guard returns',
 		content:
-			'# Control Flow\n\nStandard JavaScript control flow can contain template statements inside component bodies and nested element children.\n\n```tsx\ncomponent List({ items }: { items: string[] }) {\n  if (items.length === 0) {\n    <p>"No items"</p>\n    return;\n  }\n\n  <ul>\n    for (const item of items) {\n      <li>{item}</li>\n    }\n  </ul>\n}\n```\n\nA bare `return;` exits the current render path. A return with a value is invalid inside a TSRX component body.\n\nSource: website-tsrx/src/pages/features.tsrx#if',
+			'# Control Flow\n\nStandard JavaScript control flow can contain template statements inside component bodies and nested element children.\n\n```tsx\ncomponent List({ items }: { items: string[] }) {\n  if (items.length === 0) {\n    <p>"No items"</p>\n    return;\n  }\n\n  <ul>\n    for (const item of items; index i; key item) {\n      if (!item) continue;\n      <li>{item}</li>\n    }\n  </ul>\n}\n```\n\nA bare `return;` exits the current render path. A return with a value is invalid inside a TSRX component body.\n\nInside a component `for...of` loop, `continue` skips the current rendered iteration and is the only supported top-level loop control-flow statement. Top-level `return` and `break` are invalid inside component `for...of` loops; use `continue` for item skips, `return;` for non-loop guard exits, and `break` only for `switch` cases.\n\nComponent rendering supports `for...of` list loops. Regular `for`, `for...in`, `while`, and `do...while` loops are not supported in component template scope. Move imperative loops into a nested function, event handler, effect, or helper where normal JavaScript control flow rules apply.\n\nSource: website-tsrx/src/pages/features.tsrx#for',
 	},
 	{
 		slug: 'lazy-destructuring',

package/src/server.js

@@ -307,6 +307,7 @@ ${project_context_step}
 3. For syntax uncertainty, use \`list-sections\`, \`get-documentation\`, or read \`tsrx://docs/{slug}.md\`.
 4. Keep core TSRX advice target-neutral: component declarations, statement templates, control flow, TSX expression values, lazy destructuring, style identifiers, and submodule declarations.
 5. Use \`tsrx://targets/{target}.md\` as the handoff point for target-specific responsibilities.
+5a. In component template scope, render lists with \`for...of\`; use \`continue\` to skip an item; do not use top-level \`return\` or \`break\` inside the loop, and do not use regular \`for\`, \`for...in\`, \`while\`, or \`do...while\` loops there.
 ${file_validation_step}
 ${compile_step}
 ${authoring_step}