Skip to content

Update license to AGPL#638

Open
RobbieTheWagner wants to merge 1 commit intomainfrom
license-updates
Open

Update license to AGPL#638
RobbieTheWagner wants to merge 1 commit intomainfrom
license-updates

Conversation

@RobbieTheWagner
Copy link
Member

@RobbieTheWagner RobbieTheWagner commented Feb 8, 2026

Summary by CodeRabbit

  • Documentation

    • Updated project licensing from MIT to AGPL-3.0 for open source and non-commercial use.
    • Added commercial license option with detailed terms and pricing information.
  • Chores

    • Updated package manifest to reflect new licensing model.

@coderabbitai
Copy link

coderabbitai bot commented Feb 8, 2026

📝 Walkthrough

Walkthrough

The project's licensing model has been transitioned from MIT to AGPL-3.0 with a dual-licensing framework. The original MIT LICENSE file was removed and replaced with a comprehensive LICENSE.md documenting AGPL-3.0 terms and commercial licensing options. Supporting metadata in package.json and README.md were updated to reflect this change.

Changes

Cohort / File(s) Summary
License Files
LICENSE, LICENSE.md
Removed MIT license; added new LICENSE.md file with AGPL-3.0 dual licensing model covering open-source/non-commercial use and commercial licensing options with detailed terms.
Configuration & Documentation
package.json, README.md
Updated license field from "MIT" to "AGPL-3.0" in package.json and added LICENSE.md to files array. Added License section in README.md with AGPL-3.0 terms and commercial license reference.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 A license renewed, from MIT to AGPL so bright,
Dual paths now offered—open source or commercial right,
Terms are documented, with clarity in sight,
This rabbit hops forward with licensing done right!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title accurately and concisely describes the primary change: updating the project license from MIT to AGPL-3.0, which is reflected across all modified files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch license-updates

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 and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@LICENSE.md`:
- Around line 620-621: Replace the incorrect intro.js attribution text ("Better
introductions for websites and features with a step-by-step guide for your
projects." and "Copyright (C) 2012-2021 Afshin Mehrabani") in LICENSE.md with
the correct vue-shepherd / Shepherd.js project description and copyright
holder(s) (e.g., reference the project as "vue-shepherd / Shepherd.js" and set
the copyright to "Ship Shape / Robert Wagner" with appropriate year range),
ensuring the wording matches other license entries in the file and preserves the
license format.
🧹 Nitpick comments (3)
LICENSE.md (2)

1-1: Title references "Shepherd.js" but this is the vue-shepherd repository.

Consider updating to "vue-shepherd License" or "Shepherd.js / vue-shepherd License" for clarity, since this license file ships with the vue-shepherd npm package.


6-17: Markdown heading syntax missing for section headers.

Lines 6, 8, 11, 15, 24, 26, 30, 34, 38, 48 use emoji as pseudo-headings (e.g., 🆓 Free Use - AGPL-3.0 License) but lack ## markdown heading syntax. This means they won't render as proper headings or appear in a table of contents. This is cosmetic, but worth noting.

package.json (1)

6-6: Use the non-deprecated SPDX license identifier.

"AGPL-3.0" is a deprecated SPDX identifier (since SPDX License List v3.0). Use "AGPL-3.0-only" (for version 3 only) or "AGPL-3.0-or-later" (for version 3 or any later version) instead. npm will emit a warning on publish.

Proposed fix
-  "license": "AGPL-3.0",
+  "license": "AGPL-3.0-only",

Comment on lines +620 to +621
Better introductions for websites and features with a step-by-step guide for your projects.
Copyright (C) 2012-2021 Afshin Mehrabani
Copy link

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major

Incorrect copyright notice — appears copy-pasted from intro.js.

Lines 620–621 contain a description and copyright attribution from a different project (intro.js by Afshin Mehrabani). This should be updated to reflect the actual vue-shepherd / Shepherd.js project and its copyright holder(s) (Ship Shape / Robert Wagner).

Proposed fix
-        Better introductions for websites and features with a step-by-step guide for your projects.
-        Copyright (C) 2012-2021 Afshin Mehrabani
+        vue-shepherd - A Vue wrapper for the site tour library Shepherd.
+        Copyright (C) 2025 Ship Shape Consulting LLC
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
Better introductions for websites and features with a step-by-step guide for your projects.
Copyright (C) 2012-2021 Afshin Mehrabani
vue-shepherd - A Vue wrapper for the site tour library Shepherd.
Copyright (C) 2025 Ship Shape Consulting LLC
🤖 Prompt for AI Agents
In `@LICENSE.md` around lines 620 - 621, Replace the incorrect intro.js
attribution text ("Better introductions for websites and features with a
step-by-step guide for your projects." and "Copyright (C) 2012-2021 Afshin
Mehrabani") in LICENSE.md with the correct vue-shepherd / Shepherd.js project
description and copyright holder(s) (e.g., reference the project as
"vue-shepherd / Shepherd.js" and set the copyright to "Ship Shape / Robert
Wagner" with appropriate year range), ensuring the wording matches other license
entries in the file and preserves the license format.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant