Skip to content

fix(ce-debug): honor project commit conventions - #1849

Merged
tmchow merged 1 commit into
EveryInc:mainfrom
SomSamantray:fix/ce-debug-commit-conventions
Oct 11, 2026
Merged

tmchow merged 1 commit into
EveryInc:mainfrom
SomSamantray:fix/ce-debug-commit-conventions

Conversation

@SomSamantray

Copy link
Copy Markdown
Contributor

Both ce-debug commit flows now honor the repository’s commit format, avoiding hardcoded Conventional Commit prefixes in projects that use another style. They fall back to recent history and then a conventional fix message; explicit user overrides still take precedence.

Validation: bun run test passed (4,517 passed, 10 skipped); bun run test:skill-guards passed (798 passed, 9 skipped); bun run typecheck and bun run release:validate passed. Fresh Codex evaluations matched the fixture’s commit format in both modes; Claude was unavailable. plugin:validate was not run locally because the claude CLI is unavailable; CI covers it.

Fixes #1824.

Security Disclosure

No security-relevant changes.

Agent Disclosure

  • Model: Codex · GPT-6

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ⚠️ Failed 2026-10-10T12:20:11.298973Z d526a31 PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@tmchow
tmchow merged commit 6505f4b into EveryInc:main Oct 11, 2026
8 checks passed
@github-actions github-actions Bot mentioned this pull request Oct 11, 2026
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.

ce-debug commits use a fixed conventional message, unlike ce-commit

2 participants