Skip to content

fix(commit): keep conventional prefixes lowercase - #52

Open
noyobo wants to merge 1 commit into
cheat2001:mainfrom
noyobo:fix/commit-message-autocapitalize
Open

noyobo wants to merge 1 commit into
cheat2001:mainfrom
noyobo:fix/commit-message-autocapitalize

Conversation

@noyobo

@noyobo noyobo commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

What & why

Typing a conventional prefix such as fix: in the commit summary was rewritten to Fix: by the field's sentence capitalization. That breaks the commit-message convention, so the summary and description now set autoCapitalize="off". Spell check stays on.

This covers the working-copy commit box and the inspector message editor.

Screenshots

No layout or color change. The check is that fix: stays lowercase after leaving the field, and a misspelled word is still marked while the field is focused.

image

Checklist

  • pnpm typecheck && pnpm test pass
  • cargo fmt --check && cargo clippy --all-targets -- -D warnings && cargo test pass (if Rust changed)
  • New engine functions have integration tests in tests/git_engine.rs
  • Mutating operations go through the undo tracked() wrapper where undoable
  • Colors use design tokens only (no hex in components)
  • CLAUDE.md updated if conventions/architecture changed
  • Commits carry no trailers from coding tools (Co-authored-by: <tool>, Generated with …)

Sentence capitalization rewrote "fix:" to "Fix:" in the summary and description. Spell check stays on.

This branch has not been deployed

No deployments
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