Skip to content

license: license the workspace under MIT OR Apache-2.0 - #51

Merged
Kzoeps merged 6 commits into
mainfrom
holke/license
Nov 11, 2026
Merged

Kzoeps merged 6 commits into
mainfrom
holke/license

Conversation

@holkexyz

@holkexyz holkexyz commented Nov 10, 2026 •

Copy link
Copy Markdown
Member

Part of TECH-736.

Licenses the whole workspace under MIT OR Apache-2.0, at the user's option, as the team decided on 2026-11-04. packages/api was already dual; this brings the rest of the repository in line.

  • Replaces the root LICENSE.md with LICENSE-MIT and LICENSE-APACHE (Hypercerts Foundation). They are the same files as in packages/api.
  • Sets "license": "MIT OR Apache-2.0" in the root, docs and happyview-modules package.json files.
  • Adds a License section to the README. It also says which logos and fonts aren't covered (docs/public/brand/, docs/public/favicon.ico, docs/public/fonts/), and names where the Lexicon schemas copied into docs/sources/lexicons/ come from and keeps their licenses and copyright notices: app.bsky and com.atproto from Bluesky's atproto (MIT OR Apache-2.0), pub.leaflet from Leaflet (MIT).
  • The hypercerts-api skill is distributed without this repository (npx skills add copies only the skill directory, without the root license files). So .agents/skills/hypercerts-api/ now carries both license files (copies of the root files), and its SKILL.md declares license: MIT OR Apache-2.0 in its frontmatter. license is an optional field in the Agent Skills specification.
  • Adds a patch changeset for @hypercerts-org/happyview-modules, because its tagged source now carries the new license files. Earlier releases stay MIT. @hypercerts-org/api needs no entry: it was already dual.

Open question before merging (draft until answered): the old LICENSE.md is an MIT license naming Trezy Studios, LLC, HappyView's author, as the copyright holder. It arrived with the first commit that added files (3aeabc2; the root commit 4a5bb78 is empty). That commit contains only modules written here, so the file looks like a template copied from HappyView. Can whoever set up the workspace confirm that no code here was copied from HappyView? If some was, that code keeps Trezy Studios' MIT notice, and the notice should stay in a file next to it rather than be deleted.

No code changes.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • The project is now available under either the MIT or Apache 2.0 license, at your option. Earlier releases remain MIT-licensed.
    • License notices clarify exceptions for logos and fonts and provide attribution for included third-party schemas.
    • Copyright attribution has been updated to Hypercerts Foundation.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@vercel

vercel Bot commented Nov 10, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
api-endpoints Ready Ready Preview Nov 11, 2026 7:19am UTC

Request Review

@changeset-bot

changeset-bot Bot commented Nov 10, 2026 •

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 02d2f63

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@hypercerts-org/happyview-modules Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@coderabbitai

coderabbitai Bot commented Nov 10, 2026 •

Copy link
Copy Markdown

Explain Change →

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 8fcc6813-015f-44f5-8a80-716c760338cc
Commits

Reviewing files that changed from the base of the PR and between 73b97a0 and 02d2f63.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • Trigger review

No actionable comments were generated in the recent review.

Walkthrough

The change adds MIT and Apache-2.0 license texts, documents the project’s license scope and exceptions, and updates package metadata and a Changeset to state the dual-license terms.

Changes: Project license declaration

Layer / File(s) Summary
License texts and scope
LICENSE-APACHE, LICENSE-MIT, .agents/skills/hypercerts-api/LICENSE-APACHE, .agents/skills/hypercerts-api/LICENSE-MIT, README.md
The repository and API skill include Apache-2.0 and MIT license texts. The README states the project’s license scope, exceptions for logos and fonts, and upstream licenses and copyright details for copied Lexicon schemas.
Package declarations and release note
.agents/skills/hypercerts-api/SKILL.md, package.json, docs/package.json, happyview-modules/package.json, .changeset/license-mit-or-apache.md
The root package, docs package, happyview-modules package, and API skill frontmatter declare MIT OR Apache-2.0. The Changeset describes a patch release for happyview-modules and notes that earlier releases remain MIT.

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

Change: Other

Possibly related PRs

  • hypercerts-org/api#55: Updates font assets and adds font license files. This connects to the README’s explicit exclusion of fonts from the project license.
Merge Risk: Minimal · previous review 73b97

No merge-blocking risk was identified in the licensing changes.

Pre-merge checks: Passed 5

Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and accurately summarizes the primary change: licensing the workspace under MIT OR Apache-2.0.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
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 eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (10 skipped: 10 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

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

Recent review info

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 27f6ce68-5246-40c2-aaa4-49e293b346d1
Commits

Reviewing files that changed from the base of the PR and between 5262531 and 73b97a0.

Files selected for processing: 10

  • .agents/skills/hypercerts-api/LICENSE-APACHE
  • .agents/skills/hypercerts-api/LICENSE-MIT
  • .agents/skills/hypercerts-api/SKILL.md
  • .changeset/license-mit-or-apache.md
  • LICENSE-APACHE
  • LICENSE-MIT
  • README.md
  • docs/package.json
  • happyview-modules/package.json
  • package.json

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

…pstreams

The root LICENSE-MIT and LICENSE-APACHE are now byte-identical to the
copies in packages/api, which are the canonical texts with the
Foundation's copyright line; the first version had stray blank lines.

The README's License section now names the folders holding the logos
and fonts that the code licenses don't cover, and says where the Lexicon
schemas copied from Bluesky's atproto and from Leaflet are, with their
upstream licenses and copyright notices.

A patch changeset for @hypercerts-org/happyview-modules records the
license change in the bundle's release notes, since its tagged source
carries the root license files.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The License section opened with "The licenses cover the code", which
is narrower than the license texts: LICENSE-MIT covers the software
"and associated documentation files", and LICENSE-APACHE counts
documentation source as part of the work. Read as a statement of
scope, the sentence left the repository's prose (README, CONTRIBUTING,
AGENTS, the agent skill and its references) without a license. The
section now only names what the licenses don't cover, as the other
TECH-736 READMEs do.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The hypercerts-api skill is distributed without this repository, so an
installed copy carries neither LICENSE-MIT, LICENSE-APACHE nor the
README's License section. Its frontmatter now says
"license: MIT OR Apache-2.0", the optional field in the Agent Skills
specification, as the epds-login skill does in ePDS.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The skill is distributed without this repository: `npx skills add` copies
only the skill directory, so an installed copy had the SPDX id in its
frontmatter but no copyright notice and no license texts. MIT requires the
notice in all copies, and Apache-2.0 requires passing on a copy of the
license. Copy the root LICENSE-MIT and LICENSE-APACHE unchanged into
.agents/skills/hypercerts-api/, as ePDS does for its epds-login skill.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Kzoeps

Kzoeps commented Nov 11, 2026

Copy link
Copy Markdown
Collaborator

Trezys code arent copied. must be a copy from the initial move

@Kzoeps
Kzoeps marked this pull request as ready for review November 11, 2026 04:48
@sonarqubecloud

Copy link
Copy Markdown

@Kzoeps
Kzoeps merged commit dcd8319 into main Nov 11, 2026
9 of 10 checks passed
@Kzoeps
Kzoeps deleted the holke/license branch November 11, 2026 07:23
@hypercerts-release-bot hypercerts-release-bot Bot mentioned this pull request Nov 11, 2026

This branch was previously deployed

1 inactive deployment
Preview — 02d2f63f Deployed Nov 11, 2026 by vercel[bot]
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.

2 participants