feat: npm publish readiness — version, update-check, install scripts (#286–#289)#292
Open
flyingrobots wants to merge 8 commits intomainfrom
Open
feat: npm publish readiness — version, update-check, install scripts (#286–#289)#292flyingrobots wants to merge 8 commits intomainfrom
flyingrobots wants to merge 8 commits intomainfrom
Conversation
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
fileswhitelist,publishConfig, and version export for npm publish readiness (feat: npm install -g readiness — files field, publishConfig, --version flag #286)--versionflag and update-check notifications with hexagonal architecture and Alfred resilience (feat: update check notifications — non-blocking version detection #287)install.shanduninstall.shfor~/.local/binsupport (feat: install.sh and uninstall.sh for ~/.local/bin support #288)@neuroglyph/git-mindto@flyingrobots/git-mind(chore: transfer repo from neuroglyph to flyingrobots org #289)policyvariable lint violation (Dogfood git-mind suggest action with Claude CLI #293)Problem Statement
Package is not publishable to npm — missing files whitelist, publishConfig, version export, install scripts, and correct package name. Users cannot install via npm or run update checks.
ADR Compliance (Required)
Relevant ADR(s)
Compliance Declaration
Architecture Laws Checklist (Hard Gates)
Canonical Truth & Context
--at,--observer,--trust) or deterministically defaulted.Determinism & Provenance
Artifact Hygiene
Contracts & Compatibility
Extension/Effects Safety (if applicable)
Scope Control
Backward Compatibility
--versionflag (additive). Package renamed from@neuroglyph/git-mindto@flyingrobots/git-mind(breaking for existing installs).npm install @flyingrobots/git-mindinstead of old package name--versionflag prints version and exits. Update-check notification on CLI startup.Test Plan (Required)
Unit
npm test -- test/version.test.js test/update-check.test.jsIntegration
Determinism
npm testContract/Schema
Policy Gates
Security / Trust Impact
Performance Impact
Observability / Debuggability
Operational Notes
Linked Issues / Milestones
Reviewer Quick Verdict Block (for maintainers)
MUST (Hard Gates)
SHOULD (Quality)
Verdict