Skip to content

fix: detect dependency#79

Merged
nyaomaru merged 5 commits intomainfrom
fix/detect-dependency
Jan 25, 2026
Merged

fix: detect dependency#79
nyaomaru merged 5 commits intomainfrom
fix/detect-dependency

Conversation

@nyaomaru
Copy link
Copy Markdown
Owner

Summary

Close #77

Keep dependency updates in Chore during categorization

Description

  • Detect dependency-only updates and keep them in Chore during categorization and postprocessing
  • Refactor section handling to reduce repeated logic and improve readability
  • Centralize bullet-prefix regex into shared markdown constants
  • Cast jest.fn() to FetchMock to satisfy global.fetch’s signature
  • Keep mock implementations compatible with the fetch input types
  • Ensure mocked responses are typed as Response to quiet TS

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Improves changelog classification so dependency-only updates (e.g. chore(deps): ...) remain under Chore instead of being miscategorized under Changed, addressing issue #77.

Changes:

  • Add dependency-update title detection and use it to keep dependency bumps in Chore during category tuning and markdown postprocessing.
  • Refactor section postprocessing to reduce repeated section-handling logic and centralize the bullet-prefix regex in shared markdown constants.
  • Update Jest tests/mocks to be more type-correct around fetch mocking.

Reviewed changes

Copilot reviewed 10 out of 11 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
tests/utils/section-postprocess.test.ts Adds/updates test to verify dependency bullets are moved from Changed to Chore.
tests/utils/github-auth.test.ts Refactors fetch mocking to better match global.fetch typing.
tests/utils/dependency-update.test.ts Adds unit tests for dependency-update title detection heuristics.
tests/utils/category-tune.test.ts Adds test ensuring dependency-only updates stay in Chore after tuning.
src/utils/section-postprocess.ts Adds logic to move dependency-update bullets from Changed to Chore and refactors section parsing/assembly.
src/utils/release.ts Uses shared bullet-prefix regex constant instead of a local regex.
src/utils/dependency-update.ts Introduces dependency-update title detection utility.
src/utils/category-tune.ts Applies dependency-update detection so those titles remain in Chore during tuning.
src/constants/markdown.ts Centralizes BULLET_PREFIX_RE for reuse across utilities.
package.json Adds @jest/globals dev dependency.
pnpm-lock.yaml Lockfile updates reflecting new Jest-related dependency resolution.
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

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

nyaomaru and others added 3 commits January 25, 2026 17:38
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@nyaomaru nyaomaru merged commit fc00e8f into main Jan 25, 2026
2 checks passed
@nyaomaru nyaomaru deleted the fix/detect-dependency branch January 25, 2026 16:44
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.

Improve changelog classification: chore(deps) updates should not appear under Changed

2 participants