Skip to content

fix: clarify AGENTS.md submodule guidance#909

Merged
graphite-app[bot] merged 1 commit into
mainfrom
c/update-agentsmd
Apr 25, 2026
Merged

fix: clarify AGENTS.md submodule guidance#909
graphite-app[bot] merged 1 commit into
mainfrom
c/update-agentsmd

Conversation

@camc314

@camc314 camc314 commented Apr 25, 2026

Copy link
Copy Markdown
Contributor

Clarifies AGENTS.md guidance for typescript-go submodule commits and resets.

Copilot AI review requested due to automatic review settings April 25, 2026 09:45
@camc314 camc314 changed the title Clarify submodule guidance fix: AGENTS.md clarify submodule guidance Apr 25, 2026
@camc314 camc314 changed the title fix: AGENTS.md clarify submodule guidance fix: clarify AGENTS.md submodule guidance Apr 25, 2026
@camc314 camc314 self-assigned this Apr 25, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the AI assistant guide (AGENTS.md) to more precisely distinguish between local typescript-go submodule work, patch workflow commits, and intentional submodule pointer update commits, with added guidance around when not to reset the submodule.

Changes:

  • Rewords the critical warning to specifically prohibit committing typescript-go submodule pointer changes.
  • Adds a clearer pre-commit checklist distinguishing “submodule update” vs “patch workflow” commits.
  • Adds a “Do Not Reset Automatically” section that restricts when typescript-go/ should be reset/cleaned.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread AGENTS.md Outdated
Comment thread AGENTS.md Outdated

camc314 commented Apr 25, 2026

Copy link
Copy Markdown
Contributor Author

Merge activity

Clarifies AGENTS.md guidance for typescript-go submodule commits and resets.
@graphite-app graphite-app Bot force-pushed the c/update-agentsmd branch from d8d5c10 to 5985b4b Compare April 25, 2026 10:01
@graphite-app graphite-app Bot merged commit 5985b4b into main Apr 25, 2026
9 checks passed
@graphite-app graphite-app Bot removed the 0-merge label Apr 25, 2026
@graphite-app graphite-app Bot deleted the c/update-agentsmd branch April 25, 2026 10:08
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.

2 participants