Skip to content

Commit 30e57b4

Browse files
jitokimoh-my-graph
andcommitted
docs(changelog): quote both diagnostics — the old one, and the line it now carries
The Unreleased entry for the placeholder fix described the new sentence but never showed either message, so a reader could not tell what actually changed in the output. It now quotes both forms verbatim, and names the command they were captured from: go run ./cmd/oh-my-graph run /tmp/omg-repro-placeholder.yaml --dry-run Old (209df64^, internal/handoff/handoff.go:146 and :170): cannot resolve {{ inputs.demo }}: no such input was provided cannot resolve {{ artifacts.nosuch }}: artifact not available (its producing node has not completed) New (handoff.go:159 and :183, the quotingHint constant at :56): cannot resolve {{ inputs.demo }}: no such input was provided note: every {{ ... }} in a prompt is resolved, including one that is only being quoted or explained — to keep such text literal, break the two braces apart ("{ {"), or pass the text in as an input or artifact instead of writing it in the prompt The quoted text was read back off the running binary, not copied from the implementation report. CHANGELOG.md only — no code, no release, no tag, no version bump. Signed-off-by: jitokim <pigberger70@gmail.com> Co-Authored-By: oh-my-graph <graphs@oh-my-graph.dev>
1 parent 209df64 commit 30e57b4

1 file changed

Lines changed: 29 additions & 5 deletions

File tree

CHANGELOG.md

Lines changed: 29 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,11 +17,35 @@ oh-my-graph is **alpha software**. The graph YAML schema, the CLI, and the
1717
reasons were written as if the graph had meant the token — the artifact one
1818
asserts "its producing node has not completed" about a node that may not
1919
exist — so a prompt that only explains the syntax to the model got a
20-
diagnostic pointing at wiring that was never there. Both errors gain one
21-
line: every `{{ ... }}` in a prompt is resolved, including one only being
22-
quoted or explained; to keep such text literal, break the two braces apart
23-
(`{ {`) or pass the text in as an input or artifact. Wording only — no
24-
escaping syntax, no flag, and interpolation itself is unchanged.
20+
diagnostic pointing at wiring that was never there.
21+
22+
Before, both errors said only what was missing:
23+
24+
```
25+
cannot resolve {{ inputs.demo }}: no such input was provided
26+
cannot resolve {{ artifacts.nosuch }}: artifact not available (its producing node has not completed)
27+
```
28+
29+
After, each carries a second line stating the rule and the way out:
30+
31+
```
32+
cannot resolve {{ inputs.demo }}: no such input was provided
33+
note: every {{ ... }} in a prompt is resolved, including one that is only being quoted or explained — to keep such text literal, break the two braces apart ("{ {"), or pass the text in as an input or artifact instead of writing it in the prompt
34+
```
35+
36+
The artifacts form gains the same single line after `artifact not available
37+
(its producing node has not completed)`. The command those two captures came
38+
from is the address for them — a graph whose prompts only *mention* the
39+
syntax:
40+
41+
```sh
42+
go run ./cmd/oh-my-graph run /tmp/omg-repro-placeholder.yaml --dry-run
43+
```
44+
45+
Appended at exactly those two sites and nowhere else: a filter on a feedback
46+
token and an unreadable artifact file are real wiring bugs, where this advice
47+
would mislead. Wording only — no escaping syntax, no flag, and what
48+
`Interpolate` resolves is unchanged.
2549

2650
- **`runs list` collapses the per-run skip warnings into one summary line, and
2751
the detail moves behind `--show-skipped`.** Option chosen: *collapse by

0 commit comments

Comments
 (0)