@octanejs/mcp-server 0.2.17 → 0.2.19

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.19",
   "type": "module",
   "engines": {
     "node": ">=22.22.2"

package/skills/bridge-react-package.md

@@ -39,6 +39,7 @@ of bridging by hand:
 | `@streamdown/mermaid` | `@octanejs/streamdown/mermaid` |
 | `@streamdown/cjk` | `@octanejs/streamdown/cjk` |
 | `recharts` | `@octanejs/recharts` |
+| `react-resizable-panels` | `@octanejs/resizable-panels` |
 | `@react-three/fiber` | `@octanejs/three` |
 | `@visx/*` | `@octanejs/visx` |
 | `react-redux` | `@octanejs/redux` |
@@ -128,11 +129,7 @@ So a bridge never means "run the React package unchanged". It means:
    the official bindings:
 
    ```ts
-   import { useMemo, useRef } from 'octane';
-
-   export function subSlot(slot: symbol | undefined, tag: string) {
-   	return slot !== undefined ? Symbol.for((slot.description ?? '') + ':' + tag) : undefined;
-   }
+   import { subSlot, useMemo, useRef } from 'octane';
 
    export function useControllableState(opts, slot?: symbol) {
    	const valueRef = useRef(opts.defaultValue, subSlot(slot, 'value'));

package/src/bridge.js

@@ -27,6 +27,8 @@ export const KNOWN_BINDINGS = {
 	'@rainbow-me/rainbowkit': '@octanejs/rainbowkit',
 	'@tanstack/react-router': '@octanejs/tanstack-router',
 	'@tanstack/react-store': '@octanejs/tanstack-store',
+	'@xstate/react': '@octanejs/xstate',
+	'@xstate/store-react': '@octanejs/xstate-store',
 	'@tanstack/react-router-ssr-query': '@octanejs/tanstack-router-ssr-query',
 	'@tanstack/react-hotkeys': '@octanejs/tanstack-hotkeys',
 	'@tanstack/react-pacer': '@octanejs/tanstack-pacer',
@@ -62,6 +64,7 @@ export const KNOWN_BINDINGS = {
 	'react-dropzone': '@octanejs/dropzone',
 	sonner: '@octanejs/sonner',
 	'react-error-boundary': '@octanejs/react-error-boundary',
+	'react-resizable-panels': '@octanejs/resizable-panels',
 	'react-transition-group': '@octanejs/transition-group',
 	'react-day-picker': '@octanejs/day-picker',
 	'input-otp': '@octanejs/input-otp',
@@ -183,6 +186,7 @@ export const KNOWN_VANILLA_CORES = {
 	'@dnd-kit/react': '@dnd-kit/dom',
 	'embla-carousel-react': 'embla-carousel',
 	'@xstate/react': 'xstate',
+	'@xstate/store-react': '@xstate/store',
 	'react-redux': 'redux',
 	'@reduxjs/toolkit': 'redux',
 	'react-i18next': 'i18next',
@@ -264,11 +268,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 +281,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 +500,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)) {