Update checkout front matter for current repository#18713
Merged
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Contributor
|
@copilot review github context prompt and populate with the list of checkouts, including the current so that the agent knows where the checkouts are and what repos they are for. |
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Contributor
Author
Done in 688ab13. The GitHub context prompt now includes the full checkout list when user checkouts are configured. For each checkout, it shows the path and repository, marking the The checkout list is injected into Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Copilot
AI
changed the title
[WIP] Update checkout front matter for current repository
Update checkout front matter for current repository
Feb 28, 2026
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Contributor
Author
Done in b137b96. Updated
|
Contributor
There was a problem hiding this comment.
Pull request overview
Adds support for a current: true flag in checkout: frontmatter to identify a single “primary” repository, and enriches the GitHub context prompt with a rendered list of configured checkouts (marking the current one).
Changes:
- Adds
currentto checkout config parsing/validation, internal checkout merging, and JSON schema. - Adds
CheckoutManager.GetCurrentRepository()and prompt-building helpers to surface checkout information to the agent. - Updates docs and the smoke workflow lock file to reflect the new GitHub context content.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| pkg/workflow/unified_prompt_step.go | Injects a generated checkout list into the GitHub context prompt before expression extraction/substitution. |
| pkg/workflow/checkout_manager.go | Adds Current to config + resolved entries, parses/validates current, and generates markdown checkout-list prompt content. |
| pkg/workflow/checkout_manager_test.go | Adds unit tests for parsing/validation, GetCurrentRepository, and checkout prompt rendering. |
| pkg/parser/schemas/main_workflow_schema.json | Extends checkout schema with current: boolean. |
| docs/src/content/docs/reference/cross-repository.md | Documents current and adds a new section with examples. |
| .github/workflows/smoke-codex.lock.yml | Updates the locked prompt content to include the new checkouts section. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Contributor
|
@copilot merge main recompile |
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Contributor
Author
|
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Contributor
|
@copilot merge main and recompile Show full path in prompt and cwd |
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Contributor
Author
Done in 07fd42b. The checkout prompt now shows full absolute paths using Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Contributor
|
@copilot add checkout entry to smoke-codex and recompile |
…val; add current:true to smoke-codex checkout Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Contributor
Author
Fixed in 5bee2d3. Two issues addressed:
Done in 5bee2d3. Converted Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Contributor
|
@copilot lint go |
…d isDockerAvailable and schemaValidationLog Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Contributor
Author
Fixed in 3b8e283. Three lint issues resolved:
|
Contributor
|
🌑 The shadows whisper... Smoke Codex failed. The oracle requires further meditation... |
Contributor
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
Contributor
|
📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing... |
Contributor
Agent Container Tool Check
Result: 12/12 tools available ✅ Overall Status: PASS
|
Contributor
|
Commit pushed:
|
Contributor
|
🤖 Smoke test §22524448380 results for PR #18713:
Status: cc
|
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
This PR adds current: true checkout support to tell agents which repository is their primary target. The implementation is clean: schema update, validation, prompt injection, and tests all present. The injection approach (inserting before (/github-context)) is a reasonable pattern. Docs and smoke-codex workflow are updated accordingly. No issues found — looks good.
📰 BREAKING: Report filed by Smoke Copilot
| // "${{ github.repository }}" for any checkout that does not have an explicit repository | ||
| // configured; callers must ensure these expressions are processed by an ExpressionExtractor | ||
| // so the placeholder substitution step can resolve them at runtime. | ||
| func buildCheckoutsPromptContent(checkouts []*CheckoutConfig) string { |
Contributor
There was a problem hiding this comment.
The buildCheckoutsPromptContent function cleanly separates the prompt-building concern from the checkout manager's core responsibilities. The (cwd) marker and **current** annotation should give agents clear guidance about which repository to target. Nice design.
| unifiedPromptLog.Printf("Injecting checkout list into GitHub context (%d checkouts)", len(data.CheckoutConfigs)) | ||
| const closeTag = "</github-context>" | ||
| if idx := strings.LastIndex(combinedPromptText, closeTag); idx >= 0 { | ||
| combinedPromptText = combinedPromptText[:idx] + checkoutsContent + combinedPromptText[idx:] |
Contributor
There was a problem hiding this comment.
The injection approach using strings.LastIndex to find and insert before (/github-context) is clever and resilient. One thing worth noting: if the closing tag appears multiple times in githubContextPromptText (e.g., in code examples), this would still work correctly since LastIndex finds the last occurrence. A comment explaining this intent would help future readers.
Contributor
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
Contributor
|
💥 KAPOW! 🦸 THE SMOKE TEST AGENT WAS HERE! 🦸 💨 WHOOOOSH! Claude swooped in like a caped crusader, tested ALL the systems, and — BLAMMO! — everything checks out! 🔥 ZAAAAP! The agentic workflows live! The MCP tools respond! The builds compile!
POW! BAM! BOOM! Mission accomplished. This agent has left the building. 🦸♂️💨
|
Contributor
There was a problem hiding this comment.
💥 Automated smoke test review - all systems nominal!
💥 [THE END] — Illustrated by Smoke Claude
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
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.
Current boolfield toCheckoutConfigstructcurrent booltoresolvedCheckout; propagate inadd()GetCurrentRepository()method toCheckoutManagercurrentfield and normalizepath: "."to""incheckoutConfigFromMap()path: "."to""inadd()so""and"."always merge as the same root checkout(repository, path)pair keyscurrentpropertybuildCheckoutsPromptContent()and inject checkout list into GitHub context prompt$GITHUB_WORKSPACE-based absolute paths in prompt; mark root checkout as(cwd)currentflag in table + "Marking a Primary Repository" sectiontoken:notgithub-token:foractions/checkoutpath: .from root checkout entries in docs examples and code commentsrenderGuardPoliciesJSONfrom mergeTestCodexEngineRenderMCPConfigunit test: removepayloadSizeThresholdexpectation (removed from renderer in main merge)payloadSizeThresholdremovalcurrent: trueto smoke-codex checkout entry and recompileassert.Emptyin checkout_manager_test.go (testifylint)isDockerAvailablefrom docker_images.go; move to integration test fileschemaValidationLogfrom parser/schema_validation.go💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.
✨ PR Review Safe Output Test - Run 22524448275