Skip to content

docs(readme): add Socket, CI, release, and npm badges - #407

Merged
Vivswan merged 1 commit into
mainfrom
wt/readme-badges
Sep 22, 2026
Merged

Vivswan merged 1 commit into
mainfrom
wt/readme-badges

Conversation

@Vivswan

@Vivswan Vivswan commented Sep 22, 2026

Copy link
Copy Markdown
Owner

Before

# GitHub Settings as Code

Apply declarative repository settings from `.github/settings.yml`: ...

After

# GitHub Settings as Code

[![Socket](https://badge.socket.dev/npm/package/@vivswan/github-settings-as-code)](https://socket.dev/npm/package/@vivswan/github-settings-as-code) [![CI](https://img.shields.io/github/actions/workflow/status/Vivswan/github-settings-as-code/ci.yml?branch=main&label=CI)](https://github.com/Vivswan/github-settings-as-code/actions/workflows/ci.yml?query=branch%3Amain) [![Release](https://img.shields.io/github/v/release/Vivswan/github-settings-as-code?label=release)](https://github.com/Vivswan/github-settings-as-code/releases/latest) [![npm](https://img.shields.io/npm/v/@vivswan/github-settings-as-code?label=npm)](https://www.npmjs.com/package/@vivswan/github-settings-as-code) [![npm next](https://img.shields.io/npm/v/@vivswan/github-settings-as-code/next?label=next)](https://www.npmjs.com/package/@vivswan/github-settings-as-code?activeTab=versions)

Apply declarative repository settings from `.github/settings.yml`: ...

How

  • One line, five badges, single spaces between them: Socket, CI on main, latest release, npm latest, npm next.
  • Each badge links to its source: the Socket package page, the ci.yml runs filtered to main, the latest release, the npm package page, and the npm versions tab.

Proof

  • Docs tests: bun test test/docs green (389 pass); the link tests skip https:// targets, so the five external links add no assertion.
  • Gates: bun run lint, bun run build:check green; the README stays ASCII.
Technical details
  • Reviewer note (Socket unpinned): the badge URL names the package without a version, so it tracks the package as a whole rather than the one pre-release the pinned form would freeze on.
  • Reviewer note (no license badge): package.json declares "license": "SEE LICENSE IN LICENSE.md", which shields renders as "not identifiable".
  • Line accounting: +2/-0 in README.md (the badge row and its blank line).

BEGIN_COMMIT_OVERRIDE
docs(readme): add Socket, CI, release, and npm badges

Add one badge row directly under the README title: Socket, CI on main, latest release, npm latest, and npm next.
The Socket badge is unpinned so it tracks the package rather than one pre-release.
No license badge: the package license field is "SEE LICENSE IN LICENSE.md", which shields renders as "not identifiable".
END_COMMIT_OVERRIDE

Copilot AI balanced review requested due to automatic review settings September 22, 2026 07:29
@github-actions

github-actions Bot commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

File size check

0 over a hard cap (fails), 30 warning(s).

File Size Tier Cap
.github/scripts/release-pipeline.ts:6 156 chars warn 150
.github/scripts/release-pipeline.ts:453 151 chars warn 150
.github/scripts/release-pipeline.ts:1247 153 chars warn 150
.github/scripts/release-pipeline.ts:1 34 comment lines (header) warn 25
.github/scripts/release-pipeline.ts:449 14 comment lines warn 10
.github/workflows/post-green.yml:29 153 chars warn 150
.github/workflows/update-release-pr.yml:109 14 comment lines warn 10
docs/upgrading/v2-to-v3.md 1276 lines warn 1040
src/engine/layers.ts:217 157 chars warn 150
src/flows/settings-write.ts:142 159 chars warn 150
src/flows/settings-write.ts:30 11 comment lines warn 10
src/flows/snapshot.ts:189 13 comment lines warn 10
src/github/secret-scan.ts:42 11 comment lines warn 10
src/schema.ts:186 153 chars warn 150
src/schema.ts:197 176 chars warn 150
src/sections/contract/errors.ts:14 14 comment lines warn 10
src/sections/contract/module.ts:963 185 chars warn 150
src/sections/contract/module.ts:627 12 comment lines warn 10
src/sections/contract/module.ts:897 12 comment lines warn 10
src/sections/secret_scanning_custom_patterns/compilable-form.ts:382 161 chars warn 150
src/sections/shared/roles.ts:43 13 comment lines warn 10
src/types.ts:16 156 chars warn 150
test/docs/guides.test.ts:451 155 chars warn 150
test/e2e/generators.ts 2633 lines warn 2560
test/e2e/generators.ts:1703 166 chars warn 150
test/e2e/generators.ts:1857 161 chars warn 150
test/e2e/generators.ts:1666 12 comment lines warn 10
test/engine/execute.test.ts:617 152 chars warn 150
test/flows/merge-parity.test.ts:63 164 chars warn 150
test/scripts/auto-fix-allowlist.test.ts:8 12 comment lines warn 10

Split the file, wrap the line, shorten or exempt the comment, or list the path in .file-size-allow.local with a # reason.

5 managed file(s) skipped; repo-platform owns them.

Copilot AI 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.

Copilot review overview

🟢 Approval recommended

The documentation-only change is correct and matches the stated purpose.

Review effort: Balanced
Findings: None

What changed in this PR

Adds package health, CI, release, and npm version badges to the README.

Changes:

  • Adds five linked badges below the title.
  • Tracks CI on main, releases, and npm latest/next versions.
File Description
README.md Adds the badge row.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@Vivswan
Vivswan marked this pull request as ready for review September 22, 2026 08:52
Copilot AI review requested due to automatic review settings September 22, 2026 09:11

Copilot AI 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.

Copilot review overview

🟢 Approval recommended

The badge URLs and CI workflow path are consistent with the repository and PR description.

Review effort: Balanced
Findings: None

@Vivswan Vivswan added the merge-when-green Owner approved: merge once every gate is green label Sep 22, 2026
Add one badge row directly under the README title: Socket, CI on main, latest release, npm latest, and npm next.
The Socket badge is unpinned so it tracks the package rather than one pre-release.
No license badge: the package license field is "SEE LICENSE IN LICENSE.md", which shields renders as "not identifiable".
Copilot AI review requested due to automatic review settings September 22, 2026 09:37

Copilot AI 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.

Copilot review overview

🟢 Approval recommended

The badges use valid project resources and match the stated documentation change.

Review effort: Balanced
Findings: None

@Vivswan
Vivswan merged commit dcb742f into main Sep 22, 2026
30 checks passed
@Vivswan
Vivswan deleted the wt/readme-badges branch September 22, 2026 09:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge-when-green Owner approved: merge once every gate is green

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants