chore: remove SPDX-License-Identifier headers repo-wide - #977
Merged
Conversation
The repo is uniformly MIT — root LICENSE, all six libs/*/LICENSE.md, and every one of the 130 package.json manifests. Nothing lints, tests, or otherwise enforces the per-file header, so it was 1,300+ lines of duplicated licensing metadata that new files had to remember to carry. Removes the header from the top of every source file (.ts/.tsx/.mjs/.py/ .css/.sh/.md), and from the three generators that stamped it into generated output (both inject-env.mjs scripts and the aimock scaffold generator) so it does not come back. Also drops it from the code samples in the human-in-the-loop blog post. Left alone: docs/superpowers plans and specs, which mention the header (and the old PolyForm identifier) in prose as a record of past work. Verified: lint/test/build green for chat, langgraph, ag-ui, render, a2ui, telemetry; nx test scripts; website lint + test; verify-release-versions, check-dx-coverage, verify-angular-support, and the node:test smoke specs; all 72 touched Python files compile. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
blove
enabled auto-merge (squash)
September 3, 2026 04:31
Contributor
Contributor
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Contributor
#976 landed 23 files under examples/chat/angular while this PR was open; they were written before the header was dropped and carried it in. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Contributor
Contributor
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
1 participant
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.
Why
The repo is uniformly MIT — root
LICENSE, all sixlibs/*/LICENSE.md, and every one of the 130package.jsonmanifests. The historical carve-outs are gone:libs/chatwas relicensed back to MIT andapps/minting-serviceno longer exists.Nothing enforced the per-file header either — no eslint rule, no CI step, no line in
CONTRIBUTING.mdorAGENTS.md. It was ~1,300 lines of duplicated licensing metadata that every new file had to remember to carry, restating whatLICENSEand the manifests already say.What changed
1,249 files, 1,299 deletions, 5 insertions. Pure single-line removals, except three places worth a look:
examples/chat/angular/scripts/inject-env.mjs,examples/ag-ui/angular/scripts/inject-env.mjs, andscripts/generate-aimock-scaffold.ts(3 template literals). Without these the header grows back on the next scaffold.2026-06-04-human-in-the-loop-ag-ui-agents-in-angular.mdx(3 lines), so published samples match the tree.Left untouched:
docs/superpowers/plansandspecs. Those 134 files mention the header — and the old PolyForm identifier — in prose, as a record of the past license migrations. None carry a real header; rewriting them would falsify the history.14 Python
__init__.pyfiles are now empty, which is their idiomatic state.Verification
nx run-many -t lint(chat, langgraph, ag-ui, render, a2ui, telemetry)nx run-many -t test(same six)nx run-many -t build --configuration=production(same six)nx test scriptsnx lint website,nx test websiteverify-release-versions,check-dx-coverage,verify-angular-supportnode --test examples/chat/smoke/*.spec.mjs scripts/verify-angular-support.spec.mjs.pyfiles🤖 Generated with Claude Code