Skip to content

fix: run the registry verifier shell safely - #14

Merged
jbdevprimary merged 1 commit into
mainfrom
fix/registry-verifier-shell
Aug 24, 2026
Merged

fix: run the registry verifier shell safely#14
jbdevprimary merged 1 commit into
mainfrom
fix/registry-verifier-shell

Conversation

@jbdevprimary

Copy link
Copy Markdown
Contributor

Correct the shell quoting in the new anonymous npm-registry verifier. The command is exercised locally and the full release gate passes.

Validation: pnpm verify; 591 tests; coverage gate; package and docs smoke tests.

@jbdevprimary
jbdevprimary marked this pull request as ready for review August 24, 2026 17:11
@coderabbitai

coderabbitai Bot commented Aug 24, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 26 minutes.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: cf7f6b5b-21ed-4f7f-bea8-d301473cd1f0

📥 Commits

Reviewing files that changed from the base of the PR and between e47ea98 and 6ea56dc.

📒 Files selected for processing (1)
  • .github/workflows/release.yml

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.

@amazon-q-developer amazon-q-developer 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.

The shell quoting correction in the registry verifier is valid. The change from escaped quotes (\") to plain quotes (") within the YAML literal block properly resolves the syntax issue.


You can now have the agent implement changes and create commits directly on your pull request's source branch. Simply comment with /q followed by your request in natural language to ask the agent to make changes.

@jbdevprimary
jbdevprimary merged commit 2988ad6 into main Aug 24, 2026
6 checks passed
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