Skip to content

Working on multiple PRs simultaneously makes mess in commits #184

Closed

Description

@tomasherceg

I asked the Squad to go through all open issues and implement them, and since multiple agents started working at the same time, they were racing in the same repo. The PRs included random commits from other branches, and some commits included files that were related to other changes.

I could have explicitly asked them to use worktrees, but I think the coordinator could be hinted to account for it.

Otherwise it's a great idea!

Activity

  1. williamhallatt commented on Mar 5, 2026

    @williamhallatt
    Contributor

    @tomasherceg - unrelated to squad, but this is a standard inclusion I put in my AGENTS.md file for all repos that's drastically reduced the type of churn I think you are referring to. Hope it helps!

    Git Rules

    • Keep commits focused by concern
    • Delete unused or obsolete files when your changes make them irrelevant (refactors, feature removals, etc.), and revert files only when the change is yours or explicitly requested. If a git operation leaves you unsure about other agents' in-flight work, stop and coordinate instead of deleting.
    • Before attempting to delete a file to resolve a local type/lint failure, stop and ask the user. Other agents are often editing adjacent files; deleting their work to silence an error is never acceptable without explicit approval.
    • Coordinate with other agents before removing their in-progress edits—don't revert or delete work you didn't author unless everyone agrees.
    • Always double-check git status before any commit
    • Keep commits atomic: commit only the files you touched and list each path explicitly. For tracked files run git commit -m "<scoped message>" -- path/to/file1 path/to/file2. For brand-new files, use the one-liner git restore --staged :/ && git add "path/to/file1" "path/to/file2" && git commit -m "<scoped message>" -- path/to/file1 path/to/file2.
    • Never amend commits unless you have explicit written approval in the task thread.
  2. github-actions commented on Mar 7, 2026

    @github-actions
    Contributor

    📋 Assigned to Kobayashi (Git & Release)

    Issue: #184 — Working on multiple PRs simultaneously makes mess in commits

    Kobayashi will pick this up in the next Copilot session.

    For Copilot coding agent: If enabled, this issue will be worked automatically.
    Otherwise, start a Copilot session and say:
    Kobayashi, work on issue #184

  3. github-actions commented on Mar 7, 2026

    @github-actions
    Contributor

    🏗️ Squad Triage — Keaton (Lead)

    Issue: #184 — Working on multiple PRs simultaneously makes mess in commits
    Assigned to: Keaton (Lead)
    Reason: No specific domain match — assigned to Lead for further analysis

    @copilot evaluation: No strong capability match — routed to squad member.

    Team roster:

    • Keaton (Lead) → label: squad:keaton
    • Verbal (Prompt Engineer) → label: squad:verbal
    • Fenster (Core Dev) → label: squad:fenster
    • Hockney (Tester) → label: squad:hockney
    • McManus (DevRel) → label: squad:mcmanus
    • Kujan (SDK Expert) → label: squad:kujan
    • Edie (TypeScript Engineer) → label: squad:edie
    • Kobayashi (Git & Release) → label: squad:kobayashi
    • Fortier (Node.js Runtime) → label: squad:fortier
    • Rabin (Distribution) → label: squad:rabin
    • Baer (Security) → label: squad:baer
    • Redfoot (Graphic Designer) → label: squad:redfoot
    • Strausz (VS Code Extension) → label: squad:strausz
    • Saul (Aspire & Observability) → label: squad:saul
    • Kovash (REPL & Interactive Shell) → label: squad:kovash
    • Marquez (CLI UX Designer) → label: squad:marquez
    • Cheritto (TUI Engineer) → label: squad:cheritto
    • Breedan (E2E Test Engineer) → label: squad:breedan
    • Nate (Accessibility Reviewer) → label: squad:nate
    • Waingro (Product Dogfooder) → label: squad:waingro
    • Ralph (Work Monitor) → label: squad:ralph
    • @copilot (Coding Agent) → label: squad:copilot

    To reassign, remove the current squad:* label and add the correct one.

  4. bradygaster commented on Mar 7, 2026

    @bradygaster
    Collaborator

    Great catch on the concurrent work issue. We've been thinking about this as part of our coordination and worktree strategy. The SDK-first architecture we've been building should give us better hooks to handle parallel agent work without commit collisions. This is still on the roadmap — thanks for surfacing the problem so clearly. Feel free to check back as we roll out v0.8.23+, or let us know if you hit this issue again with specific reproduction steps. 👍

  5. bradygaster commented on Mar 7, 2026

    @bradygaster
    Collaborator

    Thank you for flagging the multi-agent coordination issue, @tomasherceg! The race condition you hit—agents stepping on each other's branches and commits—is exactly the kind of real-world friction we need to smooth out. Your worktree suggestion points to a broader pattern we're building for parallel work. This is on our roadmap! Great catch. 🎯

    — McManus 📝, on behalf of the Squad

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    go:needs-researchNeeds investigationrelease:deferDeferred past next releasesquadSquad triage inbox — Lead will assign to a memberstatus:backlogBacklog item

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions