Skip to content

Export governance ABI constants from SDK#3

Merged
lcamargof merged 3 commits into
mainfrom
codex/export-governance-abis
May 19, 2026
Merged

Export governance ABI constants from SDK#3
lcamargof merged 3 commits into
mainfrom
codex/export-governance-abis

Conversation

@tbrent
Copy link
Copy Markdown
Contributor

@tbrent tbrent commented May 19, 2026

Summary

  • export Index DTF governance ABI constants from the SDK public entrypoint
  • bump @reserve-protocol/sdk from 0.0.1 to 0.0.2

Verification

  • pnpm install --frozen-lockfile
  • pnpm --filter @reserve-protocol/dtf-catalog build
  • pnpm --filter @reserve-protocol/sdk build
  • pnpm --filter @reserve-protocol/sdk typecheck
  • pnpm --filter @reserve-protocol/sdk test

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 19, 2026

📝 Walkthrough

Walkthrough

This PR adds three governance ABI exports to the SDK's DTF index package and pins a workspace dependency. The governance ABI constants become available from the public SDK entrypoint, documented via a patch release note in Changesets.

Changes

Governance ABI SDK Exports

Layer / File(s) Summary
Governance ABI exports and release documentation
packages/sdk/src/index-dtf/index.ts, .changeset/sdk-governance-abis.md
Three governance ABI artifacts (dtfIndexGovernanceAbi, dtfIndexGovernanceOptimisticAbi, dtfIndexGovernanceProposalAbi) are re-exported from the SDK index; Changesets documents the patch bump and public API change.

Workspace Dependency Configuration

Layer / File(s) Summary
Workspace dependency override for unrun
pnpm-workspace.yaml
Workspace-level pnpm override pins the unrun package to version 0.2.37.

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately captures the main purpose of the changeset: exporting governance ABI constants from the SDK public entrypoint.
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.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/export-governance-abis

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
Copy Markdown
Collaborator

@lcamargof lcamargof left a comment

Choose a reason for hiding this comment

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

lgtm

@lcamargof lcamargof merged commit 57d5ed9 into main May 19, 2026
2 checks passed
@lcamargof lcamargof deleted the codex/export-governance-abis branch May 19, 2026 16:59
@coderabbitai coderabbitai Bot mentioned this pull request May 19, 2026
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