Skip to content

chore(repo): add MIT license file to root and all published packages - #234

Merged
designcode merged 1 commit into
mainfrom
chore/add-license-files
Aug 5, 2026
Merged

chore(repo): add MIT license file to root and all published packages#234
designcode merged 1 commit into
mainfrom
chore/add-license-files

Conversation

@designcode

@designcode designcode commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Every package.json declares "license": "MIT", but the repo had no license text — GitHub reported license: null, and published tarballs shipped the metadata without the notice (@tigrisdata/cli@3.7.0 on npm has no LICENSE).

Adds the MIT text at the repo root and in each of the eight published packages. Per-package because npm publishes package directories, not the repo, so a root-only file never reaches consumers. No manifest changes needed — npm includes LICENSE regardless of the files array.

Test plan

  • pnpm test in packages/cli — 834 passed, 214 skipped
  • biome check clean
  • pnpm pack on @tigrisdata/cli includes package/LICENSE

🤖 Generated with Claude Code


Note

Cursor Bugbot is generating a summary for commit 13ce3a7. Configure here.

Every package.json declared `"license": "MIT"`, but the repo carried no
license text: GitHub's API reported `license: null`, and published
tarballs shipped the metadata without the notice — @tigrisdata/cli@3.7.0
on npm contains no LICENSE.

Add the MIT text at the repo root and in each of the eight published
packages. MIT requires the notice accompany all copies, and npm
publishes package directories rather than the repo, so a root-only file
would never reach consumers. npm force-includes LICENSE regardless of a
package's `files` array, so no manifest changes are needed — verified by
packing @tigrisdata/cli, which now contains `package/LICENSE`.

No changeset: this changes only legal metadata in the published
tarballs, not package behaviour.

Assisted-by: Claude Opus 5 via Claude Code

Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
@greptile-apps

greptile-apps Bot commented Aug 5, 2026

Copy link
Copy Markdown

Greptile Summary

Adds the standard MIT license text at the repository root and to every published package.

  • Ensures GitHub can identify the repository license.
  • Ensures all eight package tarballs include the license notice.

Confidence Score: 5/5

The PR appears safe to merge with no actionable issues identified.

The license text matches the existing MIT metadata, covers every publishable package, and will be included in package tarballs.

Important Files Changed

Filename Overview
LICENSE Adds a standard MIT license matching the repository's existing MIT license metadata.
packages/agent-kit/LICENSE Adds the same MIT license notice to the published agent-kit package.
packages/agent-shell/LICENSE Adds the same MIT license notice to the published agent-shell package.
packages/cli/LICENSE Adds the same MIT license notice to the published CLI package.
packages/iam/LICENSE Adds the same MIT license notice to the published IAM package.
packages/keyv-tigris/LICENSE Adds the same MIT license notice to the published Keyv adapter package.
packages/react/LICENSE Adds the same MIT license notice to the published React package.
packages/storage/LICENSE Adds the same MIT license notice to the published storage SDK package.
packages/tigris/LICENSE Adds the same MIT license notice to the published tigris package.

Reviews (1): Last reviewed commit: "chore(repo): add MIT license file to roo..." | Re-trigger Greptile

@designcode
designcode merged commit 6cc7c2b into main Aug 5, 2026
3 checks passed
@designcode
designcode deleted the chore/add-license-files branch August 5, 2026 13:41
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