Repository navigation
contributor-information: add contributor-information queries - #21
Conversation
…contributor-information
🦋 Changeset detectedLatest commit: 3bc9fc9 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 44 minutes. View limit detailsLimit details: You’ve used the included review currently available. Review configuration: ⚙️ Run configuration
📒 Files selected for processing (21)
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (20)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughAdds registered queries to retrieve one contributor-information record or list records by author. The endpoints validate requests, query PostgreSQL, hydrate author profile and organization views, and support cursor pagination for listing. ChangesContributor information queries
Priority: ⬇️ Low Estimated code review effort: 3 (Moderate) | ~25 minutes Change: Feature Sequence Diagram(s)sequenceDiagram
participant Client
participant listContributorInformation
participant PostgreSQL
Client->>listContributorInformation: authors, sort direction, limit, cursor
listContributorInformation->>PostgreSQL: Filtered and cursor-paginated query
PostgreSQL-->>listContributorInformation: Contributor rows
listContributorInformation->>PostgreSQL: Load author profile and organization records
PostgreSQL-->>listContributorInformation: Author sidecar records
listContributorInformation-->>Client: Contributor views and optional next cursor
Merge Risk: ⚪ Minimal · up to No merge-blocking issue is established for the new contributor-information queries. Complete the normal checks before merging. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 57 functions across 12 files. (8 skipped: 8 unsupported.) ✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
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. Comment |
…contributor-information # Conflicts: # CONTRIBUTING.md # api/manifest.json # api/tests/unit/fixtures/http-fixtures.test.js
|
❌ The last analysis has failed. |
Scope
Adds public
org.hypercerts.claim.getContributorInformationandorg.hypercerts.claim.listContributorInformationendpoints with repeated-author filters, cursor pagination, full records, nullable publisher sidecars, and runtime-test fixtures.Validation
pnpm test:unit: 105/105 tests passed;pnpm checkandpnpm buildpassed locally.pnpm test:httpwas not run. Unit tests use local fixtures/fake adapters and do not install handlers, seed PostgreSQL, or contact HappyView. CI HTTP coverage exercises real XRPC over HTTP against the installed HappyView + PostgreSQL test environment; its result is pending from PR creation.Summary by CodeRabbit