Skip to content
This repository was archived by the owner on Mar 5, 2026. It is now read-only.

feat: rename functions#13

Merged
sergiofilhowz merged 1 commit intomainfrom
feat/rename-functions
Feb 16, 2026
Merged

feat: rename functions#13
sergiofilhowz merged 1 commit intomainfrom
feat/rename-functions

Conversation

@sergiofilhowz
Copy link
Copy Markdown
Contributor

@sergiofilhowz sergiofilhowz commented Feb 16, 2026 •
edited by coderabbitai bot
Loading

Summary by CodeRabbit

  • Refactor
    • Standardized API endpoint naming conventions to use Rust-style module paths (engine::console::* instead of engine.console.*) across console functions.
    • Optimized code formatting and module organization for improved consistency.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 16, 2026 •
edited
Loading

No actionable comments were generated in the recent review. 🎉

📝 Walkthrough

Walkthrough

A refactoring that standardizes RPC endpoint naming conventions across the Rust console bridge, converting from dot notation (e.g., engine.health.check) to Rust-style module paths (e.g., engine::health::check), updating all related bridge registrations and internal calls consistently.

Changes

Cohort / File(s) Summary
Bridge Endpoint Registration
packages/console-rust/src/bridge/functions.rs		 Updated engine RPC identifiers and stream/state/flow calls from dot notation to Rust-style module paths (e.g., engine.health.check → engine::health::check, stream.listAll → stream::list_all). Public API registration strings updated from engine.console.* to engine::console::* format for all 22 exposed console functions.
Trigger Registration
packages/console-rust/src/bridge/triggers.rs		 Updated trigger function path strings from engine.console.* to engine::console::* style, reordered trigger entries, and adjusted trigger definitions to align with new namespace syntax.
Module Organization & Logging
packages/console-rust/src/lib.rs, packages/console-rust/src/main.rs, packages/console-rust/src/server.rs		 Reordered public module declarations (pub mod server moved after pub mod bridge); reformatted logging statements with adjusted line breaks and multi-line blocks; reformatted CORS origin entries with explicit .parse().unwrap() calls.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Poem

🐰 Dotted paths now wear the double-colon crown,
From engine.check to engine::health::down!
Our bridge speaks Rust in syntactic style,
Each endpoint renamed with a rusty smile. 🦀

🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 11.11% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'feat: rename functions' accurately describes the main change—systematic renaming of RPC identifiers and function paths from dotted notation to Rust-style module paths across multiple files.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/rename-functions

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

@sergiofilhowz sergiofilhowz merged commit 167fdc8 into main Feb 16, 2026
1 check passed
@coderabbitai coderabbitai bot mentioned this pull request Feb 23, 2026
Merged
8 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

3 more reviewers

@andersonleal andersonleal andersonleal approved these changes

@ytallo ytallo ytallo approved these changes

@guibeira guibeira guibeira approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@sergiofilhowz @andersonleal @ytallo @guibeira