Skip to content

relayburn-sdk: split verbs into their own modules#300

Merged
willwashburn merged 1 commit intomainfrom
claude/sdk-module-stubs
May 5, 2026
Merged

relayburn-sdk: split verbs into their own modules#300
willwashburn merged 1 commit intomainfrom
claude/sdk-module-stubs

Conversation

@willwashburn
Copy link
Copy Markdown
Member

Summary

Pre-declares three empty module files (query_verbs.rs, ingest_verb.rs, export_verbs.rs) and wires them through lib.rs. The verbs themselves are filled in by three parallel follow-up PRs; this PR exists so each of those touches only its own file and they don't fight over lib.rs.

Part of #246.

Test plan

  • `cargo build -p relayburn-sdk` clean
  • `cargo doc --no-deps -p relayburn-sdk` clean

🤖 Generated with Claude Code

Add empty `query_verbs`, `ingest_verb`, `export_verbs` module files and
wire them through `lib.rs`. The verbs themselves land in three follow-up
PRs (#246 PR2-4); having the modules pre-declared lets those PRs touch
only their own file and avoid `lib.rs` merge conflicts.

Part of #246.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 5, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 7b2af5c9-42b7-4a39-b8cc-66380cffed09

📥 Commits

Reviewing files that changed from the base of the PR and between 3de3ae7 and c55d6fd.

📒 Files selected for processing (4)
  • crates/relayburn-sdk/src/export_verbs.rs
  • crates/relayburn-sdk/src/ingest_verb.rs
  • crates/relayburn-sdk/src/lib.rs
  • crates/relayburn-sdk/src/query_verbs.rs

📝 Walkthrough

Walkthrough

Three new verb modules—export_verbs, ingest_verb, and query_verbs—are added to relayburn-sdk as documented placeholders. Each module file contains descriptive documentation and TODOs for future implementation. Module declarations and public re-exports are wired into lib.rs.

Changes

Verb Module Scaffolding

Layer / File(s) Summary
Module Documentation
crates/relayburn-sdk/src/export_verbs.rs, crates/relayburn-sdk/src/ingest_verb.rs, crates/relayburn-sdk/src/query_verbs.rs
Three new modules defined with module-level documentation describing intended verbs (export, search, ingest, summary, session_cost, overhead, hotspots) and implementation plans. Each includes a TODO to port from JavaScript sources.
Module Integration
crates/relayburn-sdk/src/lib.rs
Private module declarations added for export_verbs, ingest_verb, and query_verbs with explanatory comments. Public re-exports via pub use make all module contents accessible from the SDK root.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

  • AgentWorkforce/burn#299: Establishes the foundational LedgerHandle and module re-export patterns that this PR extends with concrete verb module scaffolding.

Poem

🐰 Hop! Hop! New modules take their place,
Export, query, ingest embrace,
TODO notes from JS to Rust,
The SDK scaffold we now trust!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: splitting verbs into separate modules by adding three new module files and wiring them through lib.rs.
Description check ✅ Passed The description is directly related to the changeset, explaining the purpose of pre-declaring empty module files and wiring them through lib.rs to avoid conflicts in follow-up PRs.
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.

✏️ 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 claude/sdk-module-stubs

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant