fix: repair CI workflow YAML and failing AgentsMd e2e test on main#3666
Merged
Conversation
Assisted-By: Claude
Sayt-0
previously approved these changes
Jul 15, 2026
aheritier
added
area/ci
aheritier
previously approved these changes
Jul 15, 2026
Replaces machine-specific 'Instructions from: <path>' prefixes with a stable placeholder so cassettes don't break across environments. Assisted-By: Claude
dgageot
changed the title
Sayt-0
approved these changes
Jul 15, 2026
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
3 participants
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.
This PR fixes two independent breakages on
main.1. CI workflow YAML parse failure
When the lint job's "Set up Go" step was last modified, the
go-version-fileandcache-dependency-pathkeys were left at the wrong indentation level — they sat as siblings ofuses:rather than as children ofwith:. GitHub Actions could not parse the resulting YAML, so every CI run failed instantly with a "workflow file issue" error before a single job could start.The fix re-indents those two keys so they are correctly nested under
with:. No logic changes — the values are unchanged. Validated withactionlintandscripts/workflow-lint.sh.2. Failing
TestExec_Anthropic_AgentsMde2e testCommit
2e6951c6d(cache-stable dynamic prompts) changedadd_prompt_filesto prefix prompt-file content withInstructions from: <absolute path>, but the VCR cassette for this test predated that change, so the recorded Anthropic request no longer matched and the test failed with "requested interaction not found".Fix:
pkg/fake/proxy.go: the VCRDefaultMatchernow normalizes machine-specificInstructions from: <path>prefixes to a stableInstructions from: FILEplaceholder, keeping cassettes portable across machines and CI.e2e/testdata/cassettes/TestExec_Anthropic_AgentsMd.yaml: the recorded request body now includes the new prefix.Validated with
task test,task lint, andtask build— all green.