Skip to content

test(agent-runtime): harden slash command regressions (#543)#564

Merged
yacosta738 merged 1 commit into
developfrom
feature/dallay-543-slash-regression-tests
Apr 18, 2026
Merged

test(agent-runtime): harden slash command regressions (#543)#564
yacosta738 merged 1 commit into
developfrom
feature/dallay-543-slash-regression-tests

Conversation

@yacosta738
Copy link
Copy Markdown
Contributor

Related Issues

Fixes #543
Related to #527


Summary

This PR hardens slash-command regression coverage for the existing registry-backed session commands.

It adds focused transport-edge regressions for the highest-value remaining gaps only: CLI denied /resume {session_id}, gateway SSE denied /resume {session_id}, gateway SSE invalid /tldr extra args, and recognized slash handling on a gateway-facing plan-mode path. The goal is to freeze current behavior and prevent drift while the slash-command platform continues to evolve.


Tested Information

Validated with focused and full Rust runtime checks:

  • cargo test --manifest-path clients/agent-runtime/Cargo.toml cli_resume_target_without_caller_scope_preserves_denied_error_path -- --nocapture
  • cargo test --manifest-path clients/agent-runtime/Cargo.toml gateway::tests::web_chat_stream_ -- --nocapture
  • cargo fmt --manifest-path clients/agent-runtime/Cargo.toml --all -- --check
  • cargo clippy --manifest-path clients/agent-runtime/Cargo.toml --all-targets -- -D warnings
  • cargo test --manifest-path clients/agent-runtime/Cargo.toml
  • pre-push hook: Rust checks + runtime suite (3565 tests passed)

Reviewers should focus on:

  • the new CLI denial regression in clients/agent-runtime/src/main.rs
  • the new SSE regressions in clients/agent-runtime/src/gateway/mod.rs
  • the spec sync for targeted regression-hardening scope in openspec/specs/slash-command-registry/spec.md

Documentation Impact

  • Docs updated in:
    • openspec/specs/slash-command-registry/spec.md
    • openspec/changes/archive/2026-04-18-slash-command-regression-tests/
  • No docs update required because:
  • I verified the documentation matches the current behavior.

Breaking Changes

None.


Checklist

  • I have checked that there isn’t already a PR solving the same problem.
  • I have read the Contributing Guidelines.
  • I ensured my code follows the project's style guidelines.
  • I have added or updated tests that prove my fix is effective or that my feature works.
  • I have updated the documentation, or I explained above why no documentation update is needed.
  • I verified the documentation matches the current behavior.
  • I have documented any breaking changes in the Breaking Changes section.
  • I have linked the related issue (if any).

@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying corvus with  Cloudflare Pages  Cloudflare Pages

Latest commit: 31d459b
Status: ✅  Deploy successful!
Preview URL: https://44247938.corvus-42x.pages.dev
Branch Preview URL: https://feature-dallay-543-slash-reg.corvus-42x.pages.dev

View logs

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 18, 2026

Warning

Rate limit exceeded

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

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 16 minutes and 3 seconds.

⌛ 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.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 3a24ba59-2fcf-40e1-a242-5f4f885cbda6

📥 Commits

Reviewing files that changed from the base of the PR and between 91d9a05 and 31d459b.

📒 Files selected for processing (10)
  • clients/agent-runtime/src/gateway/mod.rs
  • clients/agent-runtime/src/main.rs
  • openspec/changes/archive/2026-04-18-slash-command-regression-tests/design.md
  • openspec/changes/archive/2026-04-18-slash-command-regression-tests/exploration.md
  • openspec/changes/archive/2026-04-18-slash-command-regression-tests/proposal.md
  • openspec/changes/archive/2026-04-18-slash-command-regression-tests/specs/slash-command-registry/spec.md
  • openspec/changes/archive/2026-04-18-slash-command-regression-tests/state.yaml
  • openspec/changes/archive/2026-04-18-slash-command-regression-tests/tasks.md
  • openspec/changes/archive/2026-04-18-slash-command-regression-tests/verify-report.md
  • openspec/specs/slash-command-registry/spec.md
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/dallay-543-slash-regression-tests

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.

@github-actions
Copy link
Copy Markdown
Contributor

✅ Contributor Report

User: @yacosta738
Status: Passed (12/13 metrics passed)

Metric Description Value Threshold Status
PR Merge Rate PRs merged vs closed 90% >= 30%
Repo Quality Repos with ≥100 stars 0 >= 0
Positive Reactions Positive reactions received 11 >= 1
Negative Reactions Negative reactions received 0 <= 5
Account Age GitHub account age 3094 days >= 30 days
Activity Consistency Regular activity over time 108% >= 0%
Issue Engagement Issues with community engagement 0 >= 0
Code Reviews Code reviews given to others 571 >= 0
Merger Diversity Unique maintainers who merged PRs 2 >= 0
Repo History Merge Rate Merge rate in this repo 92% >= 0%
Repo History Min PRs Previous PRs in this repo 249 >= 0
Profile Completeness Profile richness (bio, followers) 90 >= 0
Suspicious Patterns Spam-like activity detection 1 N/A

Contributor Report evaluates based on public GitHub activity. Analysis period: 2025-04-18 to 2026-04-18

@yacosta738 yacosta738 enabled auto-merge April 18, 2026 07:37
@yacosta738 yacosta738 disabled auto-merge April 18, 2026 07:37
@yacosta738 yacosta738 merged commit e45a26d into develop Apr 18, 2026
7 checks passed
@yacosta738 yacosta738 deleted the feature/dallay-543-slash-regression-tests branch April 18, 2026 07: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.

1 participant