Skip to content

chore: document agent workflow and enforce single-line release commits - #24

Merged
skevetter merged 1 commit into
mainfrom
codex/runtime-sdk-agent-guidance
Oct 8, 2026
Merged

skevetter merged 1 commit into
mainfrom
codex/runtime-sdk-agent-guidance

Conversation

@skevetter

@skevetter skevetter commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

Add SDK-specific agent guidance derived from main Devsy, covering the pinned toolchain, generation checks, protocol and process-ownership invariants, idiomatic Go, and current-head CI/review requirements.

Release PR automation now supplies the PR title and an explicitly empty squash body so merged commits remain a single line.

Validation: all prek hooks (including strict Go lint/format and protobuf checks), workflow validation, and diff checks passed. Final-head local CodeRabbit completed with zero findings. All current-head CI checks passed, and Greptile reviewed 0743b95 with confidence 5/5 and no actionable findings. Full remote CodeRabbit review completed on the same head with no actionable comments; no review threads remain. GitHub verified the commit signature as valid.

@coderabbitai

coderabbitai Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: d80e36b6-3a72-4303-bfe6-c553b48f6bf3
📥 Commits

Reviewing files that changed from the base of the PR and between 764223c and 0743b95.

📒 Files selected for processing (2)
  • .github/workflows/ci.yml
  • AGENTS.md

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The change adds repository guidance for development, validation, and merges. The release-please workflow now uses the pending release PR title as the squash merge subject and sets the merge body to empty.

Changes

Release merge guidance

Layer / File(s) Summary
Repository development and merge requirements
AGENTS.md
Adds Go toolchain and validation guidance, generated binding requirements, Go and protocol guidance, and pull request requirements. It specifies a single-line Conventional Commit subject and an empty body for regular and automated release merges.
Release PR squash merge
.github/workflows/ci.yml
The workflow retrieves the pending release PR title and passes it as the squash merge subject. It sets the merge body to empty.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 0743b

No verified merge-blocking risk remains; the title-format concern requires separate configuration evidence.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes both main changes: adding agent workflow documentation and enforcing single-line release commit subjects.
✨ Finishing Touches
✨ Simplify code
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@skevetter
skevetter force-pushed the codex/runtime-sdk-agent-guidance branch from 66ab94e to 0743b95 Compare October 8, 2026 05:32
@skevetter

Copy link
Copy Markdown
Contributor Author

@greptileai review

@greptile-apps

greptile-apps Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

[Medium risk] Changes release commit message format in CI workflow.

This PR appears safe to merge.

Summary

Adds SDK-specific agent guidance and sets an explicit subject and empty body when enabling squash merges for release PRs.

  • AGENTS.md covers the pinned tools, generation checks, protocol contracts, process cleanup, and review requirements.
  • .github/workflows/ci.yml uses the release PR title as the squash commit subject.
  • No actionable issues were found.

Reviews (1) · Last reviewed commit: "chore: document agent workflow and enfor..." · Reviewed by Greptile

@skevetter

Copy link
Copy Markdown
Contributor Author

@coderabbitai full review

@coderabbitai

coderabbitai Bot commented Oct 8, 2026 •

Copy link
Copy Markdown
✅ Action performed

Full review finished.

@skevetter
skevetter marked this pull request as ready for review October 8, 2026 07:13
@skevetter
skevetter merged commit 3759ad6 into main Oct 8, 2026
12 checks passed
@skevetter
skevetter deleted the codex/runtime-sdk-agent-guidance branch October 8, 2026 07:13
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