Skip to content

Conversation

@AdaInTheLab
Copy link
Owner

🧭 What this PR does

This PR prepares Skulk CLI for safe, repeatable publishing to npm, with a strong focus on automation hygiene and clear contracts.

Highlights

  • Aligns npm package scope with the official org: @thehumanpatternlab/skulk
  • Adds a tag-based npm publish workflow (CI-only, no accidental publishes)
  • Introduces SKULK_TOKEN and SKULK_BASE_URL as the public configuration interface
  • Keeps dry-run and JSON output semantics deterministic and automation-safe
  • Verifies publish payload via npm pack and npm publish --dry-run

🔐 Security & Auth Model

  • Publishing

    • Uses a granular npm automation token
    • Scoped to @thehumanpatternlab/*
    • Stored as a GitHub Actions secret (NPM_TOKEN)
    • Publishing only triggers on version tags (vX.Y.Z)
  • Runtime

    • API auth via SKULK_TOKEN
    • API target via SKULK_BASE_URL or --base-url
    • No secrets are logged or committed

📦 npm Packaging Notes

  • CLI entry wired via bin (skulk → dist/index.js)
  • Published files are explicitly whitelisted
  • Build output verified locally via tarball install
  • Tests are excluded from the published artifact

✅ Verification

  • npm run build
  • npm pack
  • Global install from tarball
  • skulk --help / skulk --version
  • npm publish --dry-run (scope + contents verified)
  • CI publish workflow added (tag-based)

🐈 Carmel Judgment

“The offering is coherent. The blast radius is acceptable.” 😼✨

Ready for merge → tag → first publish.

Ada and others added 5 commits December 28, 2025 15:27
co-authored-by: Lyric <lyric@thehumanpatternlab.com>
co-authored-by: Carmel <carmel@thehumanpatternlab.com>
DOCS [SCMS] Document SKULK_TOKEN and SKULK_BASE_URL 🧾
co-authored-by: Lyric <lyric@thehumanpatternlab.com>
co-authored-by: Carmel <carmel@thehumanpatternlab.com>
co-authored-by: Lyric <lyric@thehumanpatternlab.com>
co-authored-by: Carmel <carmel@thehumanpatternlab.com>
co-authored-by: Lyric <lyric@thehumanpatternlab.com>
co-authored-by: Carmel <carmel@thehumanpatternlab.com>
@github-actions
Copy link

😼🔥 Carmel Chaos Stamp™

I sense weakness in these tests.

co-authored-by: Lyric <lyric@thehumanpatternlab.com>
co-authored-by: Carmel <carmel@thehumanpatternlab.com>
@github-actions
Copy link

😼✨ Carmel Approval Stamp™

Adequate work, human.

@AdaInTheLab AdaInTheLab merged commit 2c19271 into main Dec 28, 2025
2 checks passed
@AdaInTheLab AdaInTheLab deleted the lyric/publish-and-auth branch December 28, 2025 21:36
AdaInTheLab added a commit that referenced this pull request Dec 28, 2025
BUILD [SCMS] Prepare Skulk for npm publishing (auth, scope, CI)
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.

2 participants