Conversation
Replace the retired "WAVE, Inc." / "WAVE Inc." string with the company of record, "WAVE Online, LLC", in the LICENSE/NOTICE text and package metadata this repository ships. Text-only: no code, version, or release change. The corrected text reaches registries with each package's next planned release. Go-live decision #18 (ruled 2026-09-28). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Your free trial PR review limit of 1000 PRs has been reached. Please upgrade your plan to continue using CodeAnt AI. |
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_bc8e68eb-b823-491a-87c5-2baf1e25407a) |
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (2)
Included review availability: This review used your included allowance. 0 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 1 review per hour. 📜 Recent review details⏰ Context from checks skipped due to timeout. (6)
🧰 Additional context used📓 Path-based instructions (1)Public repos carry LICENSE in the same commit as first code.📄 CodeRabbit inference engine (AGENTS.md) Files:
🔇 Additional comments (2)
📝 SummarySummary by CodeRabbit
WalkthroughThe LICENSE and NOTICE replace WAVE, Inc. with WAVE Online, LLC in copyright, developer attribution, and trademark statements. The copyright year remains 2026. ChangesLegal attribution
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~2 minutes Change: Other Merge Risk: ⚪ Minimal · up to The supplied change context shows the legal attribution updated consistently in both documents, with no identified merge-blocking risk. Architecture SummaryArchitecture risk: 🔵 Low · up to The change affects 2 systems. Changed systems: Architecture concerns Review detailsSystems and components
Before / after behavior
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
✨ Simplify code
Comment |
 |
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
ApprovabilityVerdict: Would Approve Macroscope's review found this PR approvable — This PR only corrects the named legal entity in LICENSE and NOTICE. It does not modify executable code, APIs, runtime behavior, or deployment configuration. Not approved because:
Review your spending limits in Billing settings. You can add or adjust custom eligibility rules. Learn more. |
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
No issues found across 2 files
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
You’re at about 92% of the monthly reviewed-line limit. You may want to disable incremental reviews to conserve quota. Reviews will continue until that limit is exceeded. If you need help avoiding interruptions, please contact contact@cubic.dev.
Architecture diagram
sequenceDiagram
participant LegalDocs as Legal Documents<br/>(LICENSE, NOTICE)
participant Pkg as Published Packages<br/>(npm, PyPI)
participant Trustee as License Trustee<br/>(Legal Entity)
participant Repo as Source Repository<br/>(wave-av/api-spec)
participant Registry as Package Registries<br/>(npm, PyPI)
Note over LegalDocs,Registry: Project Configuration Flow - Current State
LegalDocs->>LegalDocs: WAVE Online, LLC<br/>named as licensor
Note over LegalDocs: Copyright, "developed at",<br/>and trademark lines all<br/>reference WAVE Online, LLC
Repo->>LegalDocs: Ships legal text<br/>in source tree
LegalDocs-->>Repo: Apache 2.0 appendix<br/>+ NOTICE content
Pkg->>LegalDocs: Bundled metadata
LegalDocs-->>Pkg: License grant source
Pkg->>Registry: Published package
Note over Pkg: Same licensor name<br/>in all distribution<br/>channels
alt Compliance gate runs
Registry->>Repo: Package distribution<br/>trigger verification
Repo->>Repo: legal entity check
Repo-->>Registry: Valid licensor
end
Note over Trustee,LegalDocs: Legal entity matching<br/>terms, DMCA, and<br/>acceptable-use pages
Trustee->>LegalDocs: Confirms company of record
LegalDocs-->>Trustee: Consistent naming
🟢 WAVE BugBot — clearNo confident findings on the changed lines. Local review · $0 inference · wave-dispatch |
1 participant
What
Replaces the retired licensor string
WAVE, Inc./WAVE Inc.with the company of record,WAVE Online, LLC, in the legal text and package metadata this repository ships. Nothing else changes: no code, no version bump, no release.Files (2):
LICENSENOTICEThe wording matches wave-av/sdk-python byte for byte (Apache appendix
Copyright 2026 WAVE Online, LLC; NOTICE copyright, "developed at" and trademark lines).Why
No company called WAVE, Inc. exists. The operating company is WAVE Online, LLC: it is the entity named on wave.online's terms, acceptable-use and DMCA pages (checked live on 2026-09-30, zero matches for the old name), and it is the licensor already used by wave-av/sdk-python and by the
authorfield of the published npm packages. The old string was leftover boilerplate, and it put two different licensor names inside the same shipped packages. A license grant should name the company that actually grants it.This carries out go-live decision #18, ruled by the operator on 2026-09-28:
wave-sdkstays the canonical Python SDK, the legal entity is "WAVE Online, LLC" everywhere, and the corrected text reaches registries with each package's next planned release rather than through extra patch releases. The retired PyPI namewave-av-sdkstays yanked and is not deleted.Verification
Run on this branch (origin/main + this one commit) in a fresh clone:
npm run lint(redocly): the API description is valid (warnings unchanged; this PR touches no spec file)npm run test:ga: 35/35 passedscripts/public-repo-guard/content-policy.sh .: exit 0git grep -nIE 'WAVE,? Inc([^a-zA-Z]|$)'over the whole tree (CHANGELOG history excluded): 0 matches--local: 2retired-legal-entityfinding(s) on origin/main, 0 on this branchUndo
A text-only commit:
git revertof the merge restores the previous wording. Nothing was published, so there is nothing to roll back on any registry.🤖 Generated with Claude Code
Note
Cursor Bugbot is generating a summary for commit 844d64c. Configure here.