Skip to content

chore: add MIT LICENSE - #32

Open
krassx wants to merge 1 commit into
mainfrom
chore/add-mit-license
Open

krassx wants to merge 1 commit into
mainfrom
chore/add-mit-license

Conversation

@krassx

@krassx krassx commented Oct 17, 2026

Copy link
Copy Markdown
Contributor

This public repository had no licence file, so GitHub reports no licence. Public Bugsee repositories are MIT; this adds the same LICENSE text used in bugsee-cli. Found in the 2026-10-07 public-repo review.

🤖 Generated with Claude Code

Public Bugsee repositories are MIT licensed.

🤖 Generated with [Claude Code](https://claude.com/claude-code)
@coderabbitai

coderabbitai Bot commented Oct 17, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 5ddca893-6675-4156-bd2c-7155ec3c94a9
📥 Commits

Reviewing files that changed from the base of the PR and between f8bbd8b and 2cfef2f.

📒 Files selected for processing: 1
  • LICENSE

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.

📝 Summary

Summary by CodeRabbit

  • Documentation
    • Added an MIT License permitting use, copying, modification, distribution, sublicensing, and sale, provided the required copyright and permission notices are retained. The license also states that the software is provided without warranty and limits liability.

The change adds an MIT License with a 2026 Bugsee, Inc. copyright notice. It permits use, copying, modification, distribution, sublicensing, and sale, subject to retaining the copyright and permission notices. It also disclaims warranties and liability.

Priority: ⬇️ Low

Change: Other

Merge Risk: ⚪ Minimal · up to 2cfef

The added license text matches the canonical MIT terms and attribution. No concrete merge-blocking risk is established.

### Architecture Summary

Architecture risk: 🔵 Low · up to 2cfef

The change affects 1 system.

Changed systems: LICENSE

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — LICENSE (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

  • observed — Modified behavior in LICENSE: Added the MIT License with a 2026 Bugsee, Inc. copyright notice, notice-retention condition, and warranty and liability disclaimers.
Pre-merge checks: Passed 5
Check name Status Explanation
Description check Passed The description clearly explains that the pull request adds the MIT license file to a public repository.
Title check Passed The title clearly and concisely states that the pull request adds an MIT license file.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@claude

claude Bot commented Oct 17, 2026 •

Copy link
Copy Markdown

Claude finished @krassx's task in 18s —— View job


Review of PR #32: add MIT LICENSE

  • Read the diff and the surrounding repo (POM metadata, README)
  • Posted findings

Findings

  • P2: The POM <licenses> blocks still say Proprietary (ToS URL), in build.gradle.kts:343-348 and twice in compose-compiler-plugin/build.gradle.kts (:324-329, :362-367). Published artifacts would contradict the new MIT LICENSE. This is posted as an inline comment on LICENSE.

What I checked

  • The LICENSE text is the standard MIT text. It has no stray characters, and the file is a single 21-line addition.
  • I did not diff it against bugsee-cli's LICENSE, so I can't confirm it is byte-identical. The year (2026) and holder (Bugsee, Inc.) look reasonable.
  • I did not run a build or tests. This is a docs-only change that no code reads.

Summary

  1. Overall risk: Low. No code changes.
  2. Merge recommendation: Merge once the POM licence metadata is either updated to MIT in this PR (or a follow-up before the next release) or confirmed to be intentionally different.
  3. Most important issue: the Proprietary-vs-MIT mismatch above.
  4. Positives: The change is minimal and the text is standard. It uses the same licence as the other public Bugsee repos, and GitHub will now detect it.

Comment thread LICENSE
@@ -0,0 +1,21 @@
MIT License

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Severity: P2 Medium
Location: LICENSE:1 (conflicts with build.gradle.kts:343-348, compose-compiler-plugin/build.gradle.kts:324-329 and :362-367)
Problem: The published POM metadata still declares <license><name>Proprietary</name><url>https://bugsee.com/tos/</url></license> in all three POM blocks. This PR adds an MIT LICENSE but leaves them untouched.
Impact: The repo and its Maven Central artifacts (com.bugsee:bugsee-android-gradle-plugin and the compose compiler plugin) would then state two different licences. Consumers, SCA and licence-scanning tools, and Maven Central metadata read the POM, not the repo root file.
Scenario: The next release publishes a POM that says "Proprietary" while the source repo is MIT. Compliance scanners flag the artifact, or downstream users cannot tell which terms apply.
Fix: If MIT is intended for the published artifacts, change the three POM blocks to name.set("MIT License") and url.set("https://opensource.org/licenses/MIT"). If the artifacts are meant to stay under the ToS, say so explicitly, because the two will otherwise contradict each other. Check this with whoever owns the licensing decision. Maybe also add a licence line to README.md, which currently has none.

This branch has not been deployed

No deployments
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