🐛 fix: make .extra.tmpl actually renderable by op inject - #11
Merged
Conversation
op inject parses the entire template — comments included — and errors on any curly-brace pair or bare op:// text that isn't a real secret reference. The template's self-documenting comments contained both, so every render failed (including bootstrap's, which misreported it as an auth failure). Instructions move to CLAUDE.md where literal syntax is safe; the template keeps only a pointer and the parser warning. Verified: `op inject -i .extra.tmpl` now renders cleanly, with no auth needed while the template holds no references. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Guards the constraint that broke rendering: comment lines must never contain brace pairs or op:// text, and any real reference must be brace-wrapped with vault/item/field segments. Runs on CI without the op CLI. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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.
Summary
op injectparses the entire template — comments included — and errors on any curly-brace pair or bareop://text that isn't a real brace-wrapped secret reference..extra.tmpl's self-documenting comments contained both, so every render has always failed, including the one bootstrap runs (whose error message misattributed it to auth).tests/extra-tmpl.batsenforces the invariant on CI without needing the op CLI: no braces orop://in comments, and active references must be{{ op://vault/item/field }}-shaped.Test plan
op inject -i .extra.tmplrenders cleanly (verified locally; no auth needed while reference-free)bats tests/extra-tmpl.bats— 3/3🤖 Generated with Claude Code