Fix routing-tag accuracy in the site Concepts doc - #389
Merged
Conversation
The Concepts page had inherited the stale pre-[galaxy] routing enum -- it listed only [galaxy]/[local]/[hybrid] and dropped [remote] entirely, and it described per-step routing as bracket tags when steps actually record route, tool, and verification on sub-bullets. Reconciled against the live system prompt in extensions/loom/context.ts, which treats [galaxy] as the default and defines all four header tags ([galaxy], [remote], [hybrid], [local]). Also fixed the loom-invocation example to use the real ```loom-invocation fence with the required submitted_at field instead of a plain yaml block, noted that a completed invocation still needs output verification before the step checkbox flips, surfaced that Galaxy jobs run in the background, and corrected the git-init trigger wording. Codex made a review pass over these against context.ts.
dannon
marked this pull request as ready for review
July 19, 2026 02:26
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 free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
The Concepts page (
site/src/content/docs/concepts.md) had inherited a stale routing-tag description, so I reconciled it against the live system prompt inextensions/loom/context.ts(with a codex review pass on top).What changed:
[galaxy]/[local]/[hybrid]and dropped[remote]entirely. Now lists all four header tags ([galaxy],[remote],[hybrid],[local]) with[galaxy]as the default and the galaxy-vs-remote distinction spelled out. Also fixed the per-step format: steps recordRouting:/Tool:/Verification:sub-bullets, they don't carry bracket tags.loom-invocationexample -- was a plain```yamlblock missing the requiredsubmitted_atfield. Now uses the real```loom-invocationfence withsubmitted_at, and notes that acompletedinvocation still needs output verification before the step checkbox flips to- [x].Docs-only. Builds clean -- all 6 pages render. Companion to the agent-docs drift cleanup (same root cause, different files).