Skip to content

feat: use Leaflet documents for rich-text descriptions - #152

Merged
aspiers merged 2 commits into
developfrom
feat/leaflet-descriptions
Mar 2, 2026
Merged

feat: use Leaflet documents for rich-text descriptions#152
aspiers merged 2 commits into
developfrom
feat/leaflet-descriptions

Conversation

@holkexyz

Copy link
Copy Markdown
Member

Summary

  • Replace description (string) + descriptionFacets (array) with a single Leaflet linear document ref in activity and attachment lexicons, matching the existing pattern in collection
  • Make content optional on attachment records

Test plan

  • npm run check passes (gen-api, lint, typecheck, build, test)
  • Verify activity, attachment, and collection descriptions all use the same Leaflet pattern

🤖 Generated with Claude Code

@changeset-bot

changeset-bot Bot commented Feb 27, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 9994ea1

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/lexicon Minor

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 Feb 27, 2026

Copy link
Copy Markdown

Warning

Rate limit exceeded

@aspiers has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 8 minutes and 17 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between ee144e4 and 9994ea1.

📒 Files selected for processing (4)
  • .changeset/flat-llamas-crash.md
  • SCHEMAS.md
  • lexicons/org/hypercerts/claim/activity.json
  • lexicons/org/hypercerts/context/attachment.json
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/leaflet-descriptions

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.

@holkexyz

Copy link
Copy Markdown
Member Author

FYI I reordered some fields in the activity claim, which might look like a bigger change in the files, but it isn't:)

@aspiers
aspiers force-pushed the feat/leaflet-descriptions branch 2 times, most recently from db2cb6b to 42cfad9 Compare March 1, 2026 23:58
holkexyz and others added 2 commits March 2, 2026 00:33
- Replace description string + descriptionFacets with Leaflet linear
  document ref in activity and attachment lexicons
- Make attachment content optional

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@aspiers
aspiers force-pushed the feat/leaflet-descriptions branch from 42cfad9 to 9994ea1 Compare March 2, 2026 00:33
@aspiers
aspiers merged commit 5b3e4c7 into develop Mar 2, 2026
5 checks passed
@aspiers
aspiers deleted the feat/leaflet-descriptions branch March 2, 2026 00:37
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