Skip to content

chore: plugin declared MIT while the project is All Rights Reserved - #15

Merged
bkd-dotcom merged 1 commit into
mainfrom
chore/license-all-rights-reserved
Aug 18, 2026
Merged

chore: plugin declared MIT while the project is All Rights Reserved#15
bkd-dotcom merged 1 commit into
mainfrom
chore/license-all-rights-reserved

Conversation

@bkd-dotcom

Copy link
Copy Markdown
Member

claude-code/signetry/.claude-plugin/plugin.json declared:

"license": "MIT"

The project is All Rights Reserved. Left as-is, the plugin directory arguably shipped under MIT terms — granting rights the rest of the project explicitly reserves, and contradicting the CLA every contributor signs. Now:

"license": "Proprietary — All Rights Reserved"

Matching signetry-core's pyproject.toml wording, so the two agree.

Scope check

I looked for other MIT claims rather than assuming this was the only one:

check result
LICENSE / LICENSE.md file none in this repo
GitHub's detected license none
.claude-plugin/marketplace.json no license field
pyproject.toml / package.json not present in this repo

So this declaration was the only one. JSON re-parsed after the edit to confirm it's still valid.

The same declaration exists in Signetry/claude-code (the split-out repo) and is fixed there in a matching PR.

.claude-plugin/plugin.json declared "license": "MIT". The project is
All Rights Reserved, so left as-is the plugin directory arguably shipped
under MIT terms — granting rights the rest of the project reserves, and
contradicting the CLA every contributor signs.

Now "Proprietary — All Rights Reserved", matching signetry-core's
pyproject.toml wording.

Checked for other MIT claims: no LICENSE file exists here and GitHub
detects no license, so this declaration was the only one.
@github-actions

Copy link
Copy Markdown
Contributor

Signetry Reviewer — 🟡 Needs human review

A human should decide — the required check is unknown.

Deterministic gates (the authority)

Gate Status
Required status check — unknown
Secret scan ✅ clean
CI permission / OIDC ✅ no forbidden change
Dependency skew ✅ ok
All green

Findings

No issues found by the deterministic scanners.

Merge

A human should review and merge.

This review is advisory. It never merges on its own judgement — the deterministic gates + a human are the authority. Findings can have false negatives; a green bot verdict is not a guarantee.

@bkd-dotcom
bkd-dotcom merged commit d130a05 into main Aug 18, 2026
2 checks passed
@bkd-dotcom
bkd-dotcom deleted the chore/license-all-rights-reserved branch August 18, 2026 21:08
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