Skip to content

chore(repo): add governing repository license - #117

Open
peter-matkovski wants to merge 1 commit into
mainfrom
agent/appsec-316-license-clarity
Open

chore(repo): add governing repository license#117
peter-matkovski wants to merge 1 commit into
mainfrom
agent/appsec-316-license-clarity

Conversation

@peter-matkovski

@peter-matkovski peter-matkovski commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add a governing repository-level Stream Source Code License
  • use the exact license already shipped with the published stream_feeds package
  • link the repository README to the local license

Contributor-rights check

All human contributors are Stream staff; automated contributions are attributable to Stream-controlled tooling.

Validation

  • the new root license matches packages/stream_feeds/LICENSE byte-for-byte
  • non-license changes pass git diff --check
  • no package source or dependency metadata changed

Tracking: APPSEC-316

Summary by CodeRabbit

  • Documentation
    • Added a proprietary Stream Source Code License Agreement covering usage rights, restrictions, ownership, support, liability, and termination.
    • Added a README section linking to the repository’s license terms.

@coderabbitai

coderabbitai Bot commented Aug 1, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The repository now includes a proprietary Stream.io Source Code License Agreement. The README identifies the repository and stream_feeds package as licensed under ./LICENSE.

Changes

Repository licensing

Layer / File(s) Summary
License terms and repository reference
LICENSE, README.md
Added the Stream.io proprietary license agreement. Added a README section that links the repository and stream_feeds package to ./LICENSE.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: brazol

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely identifies the addition of the governing repository license.
Description check ✅ Passed The description explains the license change, contributor rights, validation, and tracking, but omits the template checklist and screenshots.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch agent/appsec-316-license-clarity

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@codecov

codecov Bot commented Aug 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.60%. Comparing base (29c68cd) to head (029bafa).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #117      +/-   ##
==========================================
+ Coverage   85.53%   85.60%   +0.06%     
==========================================
  Files         124      124              
  Lines        4342     4342              
==========================================
+ Hits         3714     3717       +3     
+ Misses        628      625       -3     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@peter-matkovski peter-matkovski changed the title chore: add governing repository license chore(repo): add governing repository license Aug 3, 2026
@peter-matkovski
peter-matkovski marked this pull request as ready for review August 3, 2026 10:26
@peter-matkovski
peter-matkovski requested a review from a team as a code owner August 3, 2026 10:26
@peter-matkovski
peter-matkovski enabled auto-merge (squash) August 3, 2026 10:26

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@LICENSE`:
- Around line 128-129: In the LICENSE restriction list, change the second
consecutive “(e)” label to “(f)” while preserving the surrounding license text;
apply the same correction to the canonical package license if byte-for-byte
identity is required.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: b04c9fb5-c7e6-4620-94d5-30db9989b393

📥 Commits

Reviewing files that changed from the base of the PR and between 29c68cd and 029bafa.

📒 Files selected for processing (2)
  • LICENSE
  • README.md

Comment thread LICENSE
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