Repository navigation
test(cli): drive migrate record-currency-codes in the --json stdout purity family - #28375
Merged
Merged
Conversation
…urity family discoverFamily() finds the new member (bootSchemaStack + --json flag); FAMILY did not list it, so the nightly reconciliation pin went red. Drive it with its required pair on the fixture's jp_ticket.title, reaching the INVALID_FIELD refusal face after a read-only boot. Claude-Session: https://claude.ai/code/session_01QUs3BuiMGLNsVXrDee61JM Co-authored-by: Claude <noreply@anthropic.com>
…e purity drive Measured: on the uncompiled fixture the command answers its 'No app objects are loaded' refusal after the boot, not INVALID_FIELD. Comment only. Claude-Session: https://claude.ai/code/session_01QUs3BuiMGLNsVXrDee61JM Co-authored-by: Claude <noreply@anthropic.com>
Contributor
📓 Docs Drift CheckNothing in this diff resolved to a documentable surface (no symbol, route or SDK anchor derived from 0 changed package(s)), so this run has no opinion about the docs. What this run could not see
Coarse fallback — 0 page(s) merely mention a changed package (the pre-#9192 predicate, kept for the deliberately-wide backstop): |
Contributor
Author
|
domain:devx#1 · os-zhuang · session_01QUs3BvP5Aq9USg5GtmgAAR 席位核对 — ACCEPT ·
|
This was referenced Oct 31, 2026
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.
Requested by Sam · project thread
Fixes #27114
Clause-②: no
Metadata-protocol: none
维护者速读
json-stdout-purity.e2e.test.ts的FAMILY里补上migrate record-currency-codes及其驱动参数,仅此一个测试文件。--json且启动内核,被discoverFamily()自动发现(26 个),而FAMILY只列 25 个,夜间 tier 因此变红。packages/cli/test/json-stdout-purity.e2e.test.ts新增的那一条FAMILY及其注释。cd packages/cli && OS_TEST_TIERS=nightly pnpm exec vitest run --project integration test/json-stdout-purity.e2e.test.ts,预期 80/80 通过(约 6 分钟)。What changes
migrate record-currency-codesjoinsFAMILYwith argv--object jp_ticket --field title --code USD. All three flags are required, so without them the run stops at flag parsing and never boots. It runs as a dry run (the default), which boots read-only and writes nothing.discoverFamily()is unchanged and no case is skipped.The face it reaches was measured, not assumed: on this deliberately uncompiled fixture the command answers its "No app objects are loaded" refusal after the boot, as one compact JSON document on stdout (exit 1), with the boot diagnostics (no compiled artifact, Bootstrap complete, Graceful shutdown complete) on stderr. The comment beside the entry names that face. No
PREPAREentry was needed.Tests
Nightly integration tier, local, under the verify lock, tree at merge base
22ce3d13with the cli dependency closure built.origin/mainata0f86549, which is byte-identical to22ce3d13for this file):Tests 1 failed | 76 passed (77). The one failure is the reconciliation pin, and the received-vs-expected diff is exactly+ "migrate record-currency-codes". That confirms 26 vs 25 and that no other member is missing.a8c47a44):Test Files 1 passed (1),Tests 80 passed (80). The second commit1bdda92achanges only the comment beside the entry.Acceptance notes
OS_TEST_TIERS=nightly), so PR CI does not exercise it. The next nightly onmainis its real verdict.Verification
Derived gate families (
node scripts/pm/dispatch-gates.mjs --commands, 48 commands) were run locally. 45 exited 0.--ranreconciliation: 48 derived, 46 run, 2 not measured.check:dual-build-cjs-loadsandcheck-plugin-teardown-shape --self-testexited 3 (prerequisite not met: full dist missing, and a shallow clone).check:dts-closureexited 1 only because the local closure was built withOS_SKIP_DTS=1. Its self-test passed.None of these three reads a test file, and CI runs all three.
Generated by Claude Code