Skip to content

Commit d663c2a

Browse files
committed
docs(deployment): state the per-package walk's de-duplication bound as the source states it
`content/docs/deployment/validating-metadata.mdx` said of the per-package walk that "what it reports is exactly the set the union could not see". That sentence was removed from eight code carriers by #18878 because it is false, and the two notes that replaced it forbid restating it: `packages/cli/src/commands/compile.ts` carries "Do not re-inflate that to" it, and `packages/cli/src/utils/artifact-packages.ts` records that the claim the pass is entitled to make "is narrower than" it. The page was the last place in the tree still asserting it as a claim. The page now states the bound in the source's own settled words — the set of per-package findings no union finding already carried under the same rule, `where`, message and non-top-level position — and carries the narrowness note the source wrote down so the next reader does not re-inflate it: the key is position-insensitive, not collision-proof. Claude-Session: https://claude.ai/code/session_01QCdUBjM47SxioST9z5Zwdf Co-authored-by: Claude <noreply@anthropic.com>
1 parent e233db9 commit d663c2a

1 file changed

Lines changed: 11 additions & 4 deletions

File tree

‎content/docs/deployment/validating-metadata.mdx‎

Lines changed: 11 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -607,10 +607,17 @@ matrix answers the first question only.
607607
each package's body re-read as the stack it was assembled from, handed the
608608
artifact's own `packages[]` as **resolution context** so a sibling package's
609609
object still resolves, while the collections being judged stay that package's
610-
alone. It is de-duplicated against the union run, so what it reports is exactly
611-
the set the union could not see, and every finding it raises carries a
612-
`package '<id>' —` prefix in its `where`. A project with no `packages[]` skips
613-
it: one package by definition, already judged whole by the union run.
610+
alone. It is de-duplicated against the union run, so what it reports is the
611+
set of per-package findings no union finding already carried under the same
612+
rule, `where`, message and non-top-level position — the leading
613+
`collection[N]` index is neutralised, because a package body re-bases its
614+
collections from 0 and one finding would otherwise get two keys. Every
615+
finding it raises carries a `package '<id>' —` prefix in its `where`. That
616+
key is position-insensitive, not collision-proof: two entries that render the
617+
same `where` share one, so the bound is what the key can tell apart and no
618+
wider — `packages/cli/src/utils/artifact-packages.ts` states it, and why. A
619+
project with no `packages[]` skips it: one package by definition, already
620+
judged whole by the union run.
614621

615622
The second pass is not the first one in a stricter mood — it is a different
616623
question. A rule asking *"does this stack's app navigation point at an object

0 commit comments

Comments
 (0)