Skip to content

🤖 fix: clarify simplify workflow git context#3570

Merged
ThomasK33 merged 2 commits into
mainfrom
simplify-git-context-provenance
Jun 16, 2026
Merged

🤖 fix: clarify simplify workflow git context#3570
ThomasK33 merged 2 commits into
mainfrom
simplify-git-context-provenance

Conversation

@ThomasK33

Copy link
Copy Markdown
Member

Summary

Clarifies simplify workflow Git context provenance so child review prompts explain that branch/upstream metadata comes from the reviewed parent checkout, not necessarily the spawned child agent branch. Also applies the simplify workflow's self-review cleanup findings for prompt spacing and duplicated untracked-path handling.

Background

A simplify review prompt showed a generated branch name that looked wrong from the child agent's perspective. The built-in Git actions were reporting the parent workflow checkout correctly, but the prompt did not make that provenance explicit.

Implementation

  • Add a reusable provenance note and gitContextSource: "parent-workflow-checkout" field to simplify review context JSON.
  • Include the provenance note in review, synthesis, and fix prompts.
  • Remove redundant leading newlines in prompt sections now separated by join("\n\n").
  • Deduplicate merged untracked paths from git.status and git.changedFiles.

Validation

  • Targeted workflow harness reproduced the missing provenance note before the fix and passed after the fix.
  • workflow_run simplify --help completed successfully.
  • workflow_run simplify completed and applied the cleanup commit.
  • bun .mux/workflows/simplify.js passed.
  • bunx prettier --check .mux/workflows/simplify.js passed.
  • MUX_ESLINT_CONCURRENCY=1 make static-check passed locally before pushing.

Risks

Low. This changes a project workflow prompt and small metadata helpers only; runtime Git action semantics are unchanged.


Generated with mux • Model: openai:gpt-5.5 • Thinking: xhigh • Cost: $15.69

Clarify that simplify workflow Git context is captured from the parent workflow checkout before child agent workspaces are spawned, so branch/upstream metadata refers to the reviewed parent branch.

---

_Generated with `mux` • Model: `openai:gpt-5.5` • Thinking: `xhigh` • Cost: `$9.67`_

<!-- mux-attribution: model=openai:gpt-5.5 thinking=xhigh costs=9.67 -->
Deduplicate combined untracked path metadata and remove redundant leading newlines from simplify prompt sections.

---

_Generated with `mux` • Model: `openai:gpt-5.5` • Thinking: `xhigh` • Cost: `$9.67`_

<!-- mux-attribution: model=openai:gpt-5.5 thinking=xhigh costs=9.67 -->
@ThomasK33

Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Already looking forward to the next diff.

Reviewed commit: 2a12a24762

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@ThomasK33 ThomasK33 added this pull request to the merge queue Jun 16, 2026
Merged via the queue into main with commit 9f4f61c Jun 16, 2026
22 checks passed
@ThomasK33 ThomasK33 deleted the simplify-git-context-provenance branch June 16, 2026 07:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant