From d6c2442dbe918a089d8256c2934e5a1aa943858b Mon Sep 17 00:00:00 2001 From: Paul Bakaus Date: Thu, 13 Aug 2026 22:52:44 -0400 Subject: [PATCH] Say why the flip is session-only, not just that it is The BUILD_PATH_DEFAULT line ended on a bare absolute: a flip "is never written back to the config". True wherever the line appears, since it is emitted only when a default is already recorded, but the sentence does not carry its own scope and has now been read twice as a rule that overrides new-work's one-time offer. That is the same failure the previous commit fixed in serve-question, where an unscoped "never write it" did override the offer. The directive now states the condition it depends on and names where the exception lives, so a reader who meets the line without the surrounding code cannot draw the wrong rule from it. Written with AI assistance (Claude Code). Co-Authored-By: Claude Opus 5 (1M context) --- skill/scripts/context.mjs | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/skill/scripts/context.mjs b/skill/scripts/context.mjs index 69180ebaf..203bb378e 100644 --- a/skill/scripts/context.mjs +++ b/skill/scripts/context.mjs @@ -1316,7 +1316,12 @@ function appendBuildPathDirective(parts, ctx) { for (const root of roots) { const found = readBuildPathAt(root); if (!found) continue; - parts.push(`BUILD_PATH_DEFAULT: ${found.value} (from ${found.source}). Author direction and surface rounds with this as buildPath.value and toggle: true; a flip on the page binds that session only and is never written back to the config.`); + // "Never written back" is scoped by the fact that this directive exists at + // all: it is emitted only where a value is already recorded, which is the + // case where a flip really is session-only. Saying so inline because the + // bare absolute reads as a rule that overrides new-work's one-time offer, + // which is exactly how the same wording misfired in serve-question. + parts.push(`BUILD_PATH_DEFAULT: ${found.value} (from ${found.source}). Author direction and surface rounds with this as buildPath.value and toggle: true; a flip on the page binds that session only and is never written back, because a default is already recorded here. New-work's one-time offer to record a flipped value applies only where no default exists, which is why you are not seeing this line on those projects.`); return; } }