Skip to content

chore(release): prepare validated project update - #18

Merged
santi020k merged 6 commits into
mainfrom
feat/new-version
Aug 21, 2026
Merged

chore(release): prepare validated project update#18
santi020k merged 6 commits into
mainfrom
feat/new-version

Conversation

@santi020k

Copy link
Copy Markdown
Owner

No description provided.

@github-actions github-actions Bot added dependencies Pull requests that update a dependency file ci cli docs labels Aug 21, 2026
@santi020k santi020k added the skip-changelog Skip changeset requirement for release or non-user-facing changes label Aug 21, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1e872be862

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread packages/commitprompt/src/project.ts Outdated
Comment on lines +290 to +293
const withoutKnownValidator = source
?.split('\n')
.filter(line => !isKnownCommitlintHook(line))
.join('\n')

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Preserve commands surrounding replaced Commitlint calls

When an existing commit-msg hook combines Commitlint with another command on the same line—for example, commitlint --edit "$1" && node scripts/audit.mjs—this filter removes the entire line rather than only the obsolete validator. It can silently discard unrelated checks or leave a multiline if/then block syntactically invalid, causing all subsequent commits to fail; replace the recognized invocation without deleting the surrounding shell structure.

Useful? React with 👍 / 👎.

Comment on lines +465 to +468
if (selectedActions.has('husky-hook')) {
const huskyDependencyChanged =
setDependency(manifest, 'husky', HUSKY_VERSION, true)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Preserve existing Husky catalog specifiers

In a pnpm workspace where Husky is already declared as "husky": "catalog:"—as it is in this repository—default project setup unconditionally rewrites that declaration to ^9.1.7. This bypasses the workspace's centrally managed version and makes the supposedly package-manager-aware setup report permanent drift whenever the catalog intentionally selects another version; retain an existing valid specifier or update the relevant catalog instead.

AGENTS.md reference: AGENTS.md:L9-L10

Useful? React with 👍 / 👎.

@santi020k
santi020k merged commit cd3938e into main Aug 21, 2026
8 checks passed
@santi020k
santi020k deleted the feat/new-version branch August 21, 2026 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci cli dependencies Pull requests that update a dependency file docs skip-changelog Skip changeset requirement for release or non-user-facing changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant