Commit b936ab7
committed
docs(cli): a named artifact does not fall through the resolution chain (#4110 follow-up)
The artifact resolution-priority line reads as one fall-through list, so
`--artifact` / `OS_ARTIFACT_PATH` look like they degrade to
`<cwd>/dist/objectstack.json` when they do not resolve. They never did: a named
source is used as given, without an existence check.
That gap is what let #4110's "an absent artifact is not fatal" reach a case it
was never meant to cover, so the doc now states the rule it always followed —
named is used as-is and must exist, the fall-through is for the conventional
locations only — alongside the fail-loud behaviour restored in this PR.
Worth recording where the invariant was already written down: `os start`'s
`resolveArtifactSource` deliberately does NOT exit on a missing `--artifact`,
with the comment "Defer to serve.ts which already prints a precise error". #4110
silently removed that error; this PR restores what `os start` documents it
depends on.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Ve9HidCGRGtS2UjNPUGHSV1 parent a0901f1 commit b936ab7
1 file changed
Lines changed: 10 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
323 | 323 | | |
324 | 324 | | |
325 | 325 | | |
| 326 | + | |
| 327 | + | |
| 328 | + | |
| 329 | + | |
| 330 | + | |
| 331 | + | |
| 332 | + | |
| 333 | + | |
| 334 | + | |
| 335 | + | |
326 | 336 | | |
327 | 337 | | |
328 | 338 | | |
| |||
0 commit comments