Uh oh!
There was an error while loading. Please reload this page.
docs(qa): re-quote --seed-admin flag contract in the platform checklist - #14515
Merged
baozhoutao merged 1 commit intoSep 2, 2026
Merged
Conversation
The zero-user predicate in packages/cli/src/commands/dev.ts's --seed-admin flag text changed (#14157): "only acts on a zero-user DB" became "it acts only while no account holds the seed address and no local password login exists anywhere". Two verbatim quotations of that sentence in docs/qa/platform-checklist/areas/cli.json (the :42 verify clause and the :797 knownGaps note) go stale on the day it lands. Re-quotes both fragments from dev.ts:142, unchanged otherwise. The :42 step itself is not rewritten — "never overwrites an existing account" is the half that survived the rewrite, so the step still tests what it claims to (idempotency across a restart with a changed --admin-password). Co-Authored-By: Claude Code <noreply@anthropic.com>
baozhoutao
marked this pull request as ready for review
September 2, 2026 10:07
baozhoutao
enabled auto-merge
September 2, 2026 10:07
Uh oh!
There was an error while loading. Please reload this page.
baozhoutao
deleted the
claude/issue-14350-platform-checklist-seed-admin-requote
branch
September 2, 2026 10:45
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 freeto 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.
Fixes#14350
Re-quotes the
--seed-adminflag contract indocs/qa/platform-checklist/areas/cli.jsonfrom the landed text inpackages/cli/src/commands/dev.ts:142, after #14157 changed the zero-user predicate.What moved (verbatim from
dev.ts:142, per the unlock scan on #14350):only acts on a zero-user DB→it acts only while no account holds the seed address and no local password login exists anywhereTwo sites, both quotation fragments only — surrounding prose and the
:42step itself are untouched, per the triage/unlock-scan rulings on #14350 ("never overwrites an existing account" is unchanged, so the step still tests what it claims to)::42, inside theverifystring:flag contract: 'only acts on a zero-user DB, never overwrites an existing account'→flag contract: 'it acts only while no account holds the seed address and no local password login exists anywhere, never overwrites an existing account':797, inside aknownGapsclause note:'Default: on (idempotent — only acts on a zero-user DB, never overwrites an existing account)'→'Default: on (idempotent — it acts only while no account holds the seed address and no local password login exists anywhere, never overwrites an existing account)'No other lines touched.
docs/qa/platform-checklist/areas/identity-auth.json(sibling, out of scope per the card) does not quote the old sentence verbatim — see the PR's open questions / report for what it does say.Docs-only diff (
docs/qa/**); no changeset,skip-changesetlabel applied.Generated by Claude Code
Generated by Claude Code