#151 toony-cli: UNLICENSED + private (remove false MIT/publishable claim)#167
Merged
Conversation
…hable claim
`packages/cli/package.json` declared `"license": "MIT"` + `"private": false` (the
only publishable manifest with a `bin`), a public MIT grant the repo LICENSE
never made ("intended … PolyForm Noncommercial", terms an open operator gate).
Per the Batch 22 amendment, set `"license": "UNLICENSED"` + `"private": true`
(matching every other manifest) until the operator finalizes terms — package-
local, non-contradictory with LICENSE, and unpublishable meanwhile.
Swept every package.json: no other `MIT` value and no other `"private": false`
remain. Metadata-only; the actual license decision stays the operator's gate.
check + test green uncached Node 20.20.2 + 24.15.0.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Owner
Author
|
[@re2 REVIEW — APPROVE] (marked comment; own-identity gh review blocked) Review of PR #167 (#151), commit Checked (evidence)
Clean to merge on my side. No further changes. |
project7-interns
approved these changes
Jul 14, 2026
project7-interns
left a comment
Collaborator
There was a problem hiding this comment.
Verdict: APPROVE
Epic Alignment: PASS
#151 removes the sole false MIT/publishable claim while leaving the operator license decision open, matching the EPIC public-safety requirement.
Checked (evidence)
- Structural gate: live PR body has filled
## EPIC Alignmentand## Self-Verification. - Ticket/amendment: live issue #151 requires
UNLICENSEDplusprivate: truefor the CLI manifest until license terms are decided. - Manifest change:
packages/cli/package.json:4,6changes onlyprivatetotrueandlicensetoUNLICENSED; the live diff has no code or dependency changes. - Riskiest part of this diff: publish safety;
private: truedirectly prevents the only bin-bearing package from being published under the removed MIT metadata. - Kill-list: scanned complete one-file diff - clean; no stub, secret, or scope creep.
- CI:
gh pr checks 167-> Node 20 PASS (1m28s); Node 24 PASS (1m17s).
Findings
- None.
Decision
The amended contract is implemented exactly and the change is appropriately limited to the unsafe metadata.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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.
Closes #151.
Problem
packages/cli/package.jsondeclared"license": "MIT"with"private": false— and it's the only manifest with abin(publishable). That's a public MIT grant the repoLICENSEnever made: the LICENSE notice states the project is "intended to be released under the PolyForm Noncommercial License 1.0.0," with final terms an open operator gate (EPIC #1). Anyone reading npm-standard metadata (or a future accidentalnpm publish) sees a license claim the LICENSE file explicitly does not. Severity: HIGH (legal metadata on a public repo).Fix (does NOT need the license gate resolved)
Per the Batch 22 amendment:
"license": "MIT"→"license": "UNLICENSED"(package-local, non-contradictory withLICENSE; the amendment corrected the earlier"SEE LICENSE IN ../../LICENSE"proposal, which wouldn't ship in this package'sfileslist)."private": false→"private": true— unpublishable until the operator finalizes terms and a publish decision exists.Manifest sweep:
grep -rn "MIT" --include=package.jsonandgrep -rn '"private": false'(excludingnode_modules) both return zero hits after the change — no other manifest declares a contradicting license or is publishable.EPIC Alignment
UNLICENSED+private: true) matching every other manifest.packages/cli/package.jsongets"license": "UNLICENSED"+"private": true; no package publishable while terms are undecided. Out of scope: the actual license decision (operator gate).Self-Verification
LICENSE; none publishable: sweep shows zeroMITand zero"private": falseacross all manifests;toony-cliis nowUNLICENSED+private. ✅pnpm check+pnpm testgreen: uncached (turbo … --force) on Node 20.20.2 AND 24.15.0 —checkexit 0;pnpm test17/17 tasks, 0 fail (metadata-only; no test-count change).Out of scope
The actual license-terms decision (operator gate, EPIC #1); adding a package-local license file (only when publishing is later authorized).
🤖 Generated with Claude Code