Skip to content

chore: complete the third-party notices before the repo goes public - #1

Merged
t3dotgg merged 1 commit into
mainfrom
release-notices
Oct 7, 2026
Merged

t3dotgg merged 1 commit into
mainfrom
release-notices

Conversation

@t3dotgg

@t3dotgg t3dotgg commented Oct 7, 2026 •

Copy link
Copy Markdown
Member

The license audit before going public found 3 gaps in NOTICE.md and some stale metadata. None of them blocks MIT, but they should be fixed before the repo is public.

Changes

  • NOTICE.md now names:
    • the copied AST schema in tools/ts_ast_codegen/spec (TypeScript, Apache-2.0)
    • metaModelSchema.mts (microsoft/vscode-languageserver-node, MIT), with its license in licenses/
    • the Unicode and CLDR tables in gostd/data (Unicode License v3), with its license in licenses/
  • npm: the tsc-rs packages' NOTICE.txt also includes the Unicode license, because the binary contains those tables.
  • Metadata: Cargo.toml and npm/wasm/package.json say MIT AND Apache-2.0, as the tsc-rs packages already do. The wasm package's repository URL is pingdotgg/ts-rust (it was the old t3dotgg URL).
  • Cleanup: removes the committed empty .tmp-cargo/ts-rust-cargo-*.lock and ignores /.tmp-cargo/.

The audit found no copyleft code, no unlicensed code and no third-party project source in the repo or its history.


Made by Claude Opus 5.5 (1M context) in Claude Code, through T3 Code.

🤖 Generated with Claude Code

Note

Complete third-party license notices and switch license to MIT AND Apache-2.0

  • Changes the workspace and WASM npm package license from MIT to MIT AND Apache-2.0 in Cargo.toml and npm/wasm/package.json
  • Documents third-party provenance in NOTICE.md: the TypeScript AST schema, the Language Server Protocol meta model, and Unicode/CLDR data
  • Adds full license texts for Unicode (v3) and vscode-languageserver-node under licenses/
  • Includes the Unicode license in npm package output via portNotice in npm/pack.mjs
  • Removes the temporary Cargo lock asset and ignores /.tmp-cargo/ in .gitignore
  • Behavioral Change: npm packages now ship the Unicode license text, and published license metadata reads MIT AND Apache-2.0

Macroscope summarized f48bb0e.

- NOTICE.md names the copied AST schema (TypeScript, Apache-2.0), the LSP
  meta model (vscode-languageserver-node, MIT) and the Unicode and CLDR
  tables of gostd (Unicode License v3), with their license texts in
  licenses/. The tsc-rs NOTICE.txt also carries the Unicode license.
- Cargo.toml and the wasm package say MIT AND Apache-2.0, as the tsc-rs
  packages do. The wasm package names pingdotgg/ts-rust.
- Removes a stray .tmp-cargo lock file and ignores the dir.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 7, 2026

Copy link
Copy Markdown

Review in Change Stack →

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Team
  • Run ID: 2bcccaa3-d4a7-445b-a7b9-4a4ee84d9b15
📥 Commits

Reviewing files that changed from the base of the PR and between c007ead and f48bb0e.

⛔ Files ignored due to path filters (1)
  • .tmp-cargo/ts-rust-cargo-2193573470.lock is excluded by !**/*.lock
📒 Files selected for processing (7)
  • .gitignore
  • Cargo.toml
  • NOTICE.md
  • licenses/Unicode-LICENSE.txt
  • licenses/vscode-languageserver-node-LICENSE.txt
  • npm/pack.mjs
  • npm/wasm/package.json
 __________________________________
< Pulling the bugs out of the hat. >
 ----------------------------------
  \
   \   (\__/)
       (•ㅅ•)
       /   づ
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@t3dotgg
t3dotgg merged commit cad16b0 into main Oct 7, 2026
1 of 2 checks passed
t3dotgg added a commit that referenced this pull request Oct 7, 2026
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@t3dotgg
t3dotgg deleted the release-notices branch October 7, 2026 05:34
t3dotgg added a commit that referenced this pull request Oct 7, 2026
checker-port holds R177's crates (the same tree as goport-int48, merged
above); main is ahead of it by Theo's PRs #1, #3, #4, #5, so main's tree
is kept, as in the R176 checker-port merge.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
t3dotgg added a commit that referenced this pull request Oct 7, 2026
goport-int50 975517e: on main 22745cf (R178 plus Theo PRs #1, #3, #4, #5, #6, #7, #9), merges of goport-effectfix1 4c618c1, goport-effectfix2 3508d5e, goport-loadcrit1 e5f234e, goport-followups31 7d2cef9 and goport-effectapi1 0899a4f, a lib blob commit, a root comment commit and the build info fix
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