fix: React migrate path remapping, UI smoke test, and skill routing - #56
Merged
Conversation
aswin-shakthic
requested review from
rapchic and
shivakumar-ashwin
as code owners
July 29, 2026 15:33
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
Step 7b covers fdk run UI verification; Step 8 again holds meta-init/meta-update so agents run metrics write only after smoke test passes. Co-authored-by: Cursor <cursoragent@cursor.com>
shivakumar-ashwin
approved these changes
Jul 30, 2026
shivakumar-ashwin
added a commit
that referenced
this pull request
Jul 30, 2026
* chore: prepare 1.3.0 changelog Document React Meta default UI, installer fixes, test suite restructure, and CI/meta-init changes since v1.2.0. Co-authored-by: Cursor <cursoragent@cursor.com> * 1.3.0 * chore: document PR #56 in 1.3.0 changelog Note React migrate route remapping, UI smoke test, and skill routing updates. Co-authored-by: Cursor <cursoragent@cursor.com> --------- Co-authored-by: Ashwin Shivakumar <ashwin.shivakumar@freshworks.com> Co-authored-by: Cursor <cursoragent@cursor.com>
22 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Follow-up fixes from React Meta (MP-44662) manual testing — addresses migrate UI failures and unclear skill handoff for review/publish.
Changes:
/fdk-react-migrate— require/app/...route remapping, multi-location HTML checks, and mandatoryfdk runUI smoke test before marking migrate completejs-to-react-migration-checklist.md— same migrate gatespost-generation-message.txt+SKILL.md— explicit routing: review → fw-review, pack/publish → fw-publish (not generic AI review or raw FDK docs)Related: builds on PR #55 (React Meta default UI stack).
Type of change
SKILL.md, command file, rule file).claude-plugin,.cursor-plugin,.codex-plugin,skills/*/.cursor-plugin)installer/)tests/,installer/tests/)Test checklist
Installer tests (required when
installer/changed)installer/unchanged)Static + regex evals (required for all PRs)
LLM evals (required when
SKILL.md, a command file, or.meta.template.jsonis modified)See When to run which layer in
tests/TESTING.md.tests/all-tests-report.htmlRepo hygiene (when applicable)
AGENTS.md— N/A (no rules/commands inventory change)tests/TESTING.md— N/A (no scenario count change)docs/engine-matrix.md— N/Anode scripts/bump-version.mjs— N/ASecurity
Not applicable
installer/— installer test checklist skippedCI (runs on PR automatically)
tests/npm testinstaller/npm test(Node 24)installer/+tests/lockfilesLLM evals and
tests/e2e/e2e.share local / agent session only — not in CI.Eval report
Regex eval only for this PR (+1 test:
fdk-react-migrate requires route remapping and fdk run smoke test). Full LLM eval not re-run.