@octanejs/mcp-server 0.2.17 → 0.2.18

package/README.md

@@ -121,7 +121,7 @@ Returns a skill by name. Bundled skills (shipped with this package):
 - `setup-ssr`: server rendering and hydration setup.
 
 When running inside the octane monorepo, the skills from `.rulesync/skills` are
-also available: `authoring-tsrx`, `react-library-port`, `bug-hunter`,
+also available: `authoring-tsrx`, `octane-react-library-port`, `bug-hunter`,
 `create-a-pr`, `handle-issue`, `octane-core-extend`, `triage`,
 `performance-audit`. A test compares this map against the directory in both
 directions, so a new skill cannot stay unreachable here. This tool reads the RuleSync source, and
@@ -162,7 +162,7 @@ one manifest suite by name (`js-framework`, `todomvc`, `weather-app`,
 `hydration-interactivity`, `hydration-stress`, `lifecycle-memory`,
 `controlled-form`, `external-store-fanout`, `external-store-integrations`,
 `scheduler-responsiveness`, `suspense-recovery`, `event-delegation`,
-`application-composition`, `scaling-curves`, `streaming-ssr`,
+`application-composition`, `scaling-curves`, `activity`, `streaming-ssr`,
 `streaming-backpressure`, `compiler-throughput`, `codegen-size`, `hook-memo`,
 `bundle-size`, `bundle-reachability`, `three-renderer`, `three-bundle-size`, …)
 or every suite with `all`; `quick` selects the reduced-iteration smoke pass. The

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@octanejs/mcp-server",
-  "version": "0.2.17",
+  "version": "0.2.18",
   "type": "module",
   "engines": {
     "node": ">=22.22.2"

package/src/bridge.js

@@ -264,11 +264,11 @@ export const REACT_API_MAP = {
 		note: "Supported. Accepts React's { default } module shape and additionally a bare component from the loader; wrapping Suspense or ViewTransition in lazy() is valid (nested lazy wrappers are not).",
 	},
 	Component: {
-		status: 'unsupported',
+		status: 'rewrite',
 		note: 'No class components. Rewrite as a function component.',
 	},
 	PureComponent: {
-		status: 'unsupported',
+		status: 'rewrite',
 		note: 'No class components. Rewrite as a function component with memo.',
 	},
 	StrictMode: {
@@ -277,7 +277,7 @@ export const REACT_API_MAP = {
 	},
 	Profiler: { status: 'unsupported', note: 'Not present.' },
 	SuspenseList: { status: 'unsupported', note: 'Not present.' },
-	findDOMNode: { status: 'unsupported', note: 'Removed in React 19 too. Use refs.' },
+	findDOMNode: { status: 'rewrite', note: 'Removed in React 19 too. Use refs.' },
 	renderToString: {
 		status: 'rewrite',
 		note: 'Use renderToString() from octane/server (sync) or prerender() from octane/static (async, awaits Suspense); both return { html, css }.',
@@ -496,8 +496,8 @@ function apiRows(totals) {
 }
 
 function verdictFor(rows, classComponents) {
-	if (classComponents || rows.some((row) => row.status === 'unsupported')) return 'needs-rework';
-	if (rows.some((row) => row.status === 'rewrite' || row.status === 'partial')) {
+	if (rows.some((row) => row.status === 'unsupported')) return 'needs-rework';
+	if (classComponents || rows.some((row) => row.status === 'rewrite' || row.status === 'partial')) {
 		return 'bridgeable-with-rewrites';
 	}
 	return 'bridgeable';

package/src/bridge.test.js

@@ -166,7 +166,7 @@ describe('bridgeReport', () => {
 		expect(report.plan.join('\n')).toContain('octane/server');
 	});
 
-	it('reports class components as needs-rework', async () => {
+	it('reports class components as bridgeable with mandatory rewrites', async () => {
 		const root = await mkdtemp(join(tmpdir(), 'octane-bridge-'));
 		await writeFakePackage(root, 'classy', {
 			'index.js': `
@@ -176,6 +176,20 @@ describe('bridgeReport', () => {
 		});
 		const report = await bridgeReport({ packageName: 'classy', projectRoot: root });
 		expect(report.classComponents).toBe(true);
+		expect(report.apis.find((row) => row.name === 'Component').status).toBe('rewrite');
+		expect(report.verdict).toBe('bridgeable-with-rewrites');
+	});
+
+	it('reserves needs-rework for a public React API with no Octane rewrite', async () => {
+		const root = await mkdtemp(join(tmpdir(), 'octane-bridge-'));
+		await writeFakePackage(root, 'profiler-lib', {
+			'index.js': `
+				import { Profiler } from 'react';
+				export { Profiler };
+			`,
+		});
+		const report = await bridgeReport({ packageName: 'profiler-lib', projectRoot: root });
+		expect(report.apis.find((row) => row.name === 'Profiler').status).toBe('unsupported');
 		expect(report.verdict).toBe('needs-rework');
 	});
 
@@ -234,13 +248,13 @@ describe('bridgeReportFromSource', () => {
 		expect(report.plan.join('\n')).toContain('forwardRef');
 	});
 
-	it('reports class components as needs-rework', () => {
+	it('reports class components as bridgeable with mandatory rewrites', () => {
 		const report = bridgeReportFromSource(`
 			import React from 'react';
 			export class Panel extends React.Component { render() { return null; } }
 		`);
 		expect(report.classComponents).toBe(true);
-		expect(report.verdict).toBe('needs-rework');
+		expect(report.verdict).toBe('bridgeable-with-rewrites');
 		expect(report.plan.join('\n')).toContain('function component');
 	});
 

package/src/index.js

@@ -32,7 +32,9 @@ export const REPO_SKILLS = {
 	'handle-issue': '.rulesync/skills/handle-issue/SKILL.md',
 	'octane-core-extend': '.rulesync/skills/octane-core-extend/SKILL.md',
 	'performance-audit': '.rulesync/skills/performance-audit/SKILL.md',
+	// Temporary compatibility alias for callers using the pre-rename skill name.
 	'react-library-port': '.rulesync/skills/react-library-port/SKILL.md',
+	'octane-react-library-port': '.rulesync/skills/octane-react-library-port/SKILL.md',
 	triage: '.rulesync/skills/triage/SKILL.md',
 };
 
@@ -66,6 +68,7 @@ export const BENCHMARK_SUITES = [
 	'scaling-curves',
 	'store-selector-fanout',
 	'hook-store-composition',
+	'activity',
 	'effectful-list',
 	'list-clear',
 	'memo-wall',
@@ -295,7 +298,7 @@ export function engineeringPlanFor(input, repoMode = false) {
 			'Record what parity cannot reach as a divergence in UPSTREAM.md and status.json, with the reason, what the consumer should do instead, and a behavioral test pinning the Octane behavior.',
 			"Run the pinned release's own suite as the parity oracle: its framework-neutral tests unmodified against the reused core, its React-binding tests ported case by case with the upstream case names and citations. Record every upstream test file as run as-is, ported, or out of scope with the reason, and never weaken an upstream assertion to make it pass.",
 		];
-		if (repoMode) plan.requiredSkills.push('react-library-port');
+		if (repoMode) plan.requiredSkills.push('octane-react-library-port');
 	}
 	if (scope === 'framework-core' && !repoMode) {
 		plan.blockingConditions = [

package/src/index.test.js

@@ -165,7 +165,7 @@ describe('@octanejs/mcp-server helpers', () => {
 			true,
 		);
 
-		expect(plan.requiredSkills).toContain('react-library-port');
+		expect(plan.requiredSkills).toContain('octane-react-library-port');
 		expect(plan.gates.parity.join('\n')).toContain('packages/<name>/UPSTREAM.md');
 		expect(plan.gates.parity.join('\n')).toContain('divergence');
 		expect(plan.gates.parity.join('\n')).toContain("pinned release's own suite");
@@ -175,7 +175,7 @@ describe('@octanejs/mcp-server helpers', () => {
 			true,
 		);
 		expect(applicationPlan.gates.parity).toBeUndefined();
-		expect(applicationPlan.requiredSkills).not.toContain('react-library-port');
+		expect(applicationPlan.requiredSkills).not.toContain('octane-react-library-port');
 	});
 
 	it('blocks framework-core plans when maintainer tools are unavailable', () => {
@@ -283,6 +283,7 @@ describe('@octanejs/mcp-server helpers', () => {
 			.map((entry) => entry.name)
 			.sort();
 
+		expect(REPO_SKILLS['react-library-port']).toBe('.rulesync/skills/react-library-port/SKILL.md');
 		expect(Object.keys(REPO_SKILLS).sort()).toEqual(onDisk);
 
 		for (const file of Object.values(REPO_SKILLS)) {