Skip to content

feat(cli): proxy unrecognized commands to the project binary - #665

Open
ElijahAhianyo wants to merge 7 commits into
masterfrom
elijah/cot-cli-proxy/stack-3-proxy
Open

ElijahAhianyo wants to merge 7 commits into
masterfrom
elijah/cot-cli-proxy/stack-3-proxy

Conversation

@ElijahAhianyo

@ElijahAhianyo ElijahAhianyo commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Description

This is a build-up on #664, which laid the foundation for discovering, building, and caching metadata about a project binary. This PR only connects that logic to the CLI parsing mechanism. We forward any unrecognized command to the binary

Type of change

  • Bug fix
  • New feature
  • Documentation
  • Refactor / cleanup
  • Performance improvement
  • Other (describe above)

@github-actions github-actions Bot added the C-cli Crate: cot-cli (issues and Pull Requests related to Cot CLI) label Sep 3, 2026
@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown

🐰 Bencher Report

Projectcot
Branchelijah/cot-cli-proxy/stack-3-proxy
Testbedgithub-ubuntu-latest
Click to view all benchmark results
BenchmarkLatencyBenchmark Result
microseconds (µs)
(Result Δ%)
Upper Boundary
microseconds (µs)
(Limit %)
empty_router/empty_router📈 view plot
🚷 view threshold
14,161.00 µs
(+45.67%)Baseline: 9,721.26 µs
18,334.29 µs
(77.24%)
json_api/json_api📈 view plot
🚷 view threshold
1,112.40 µs
(+8.20%)Baseline: 1,028.07 µs
1,355.82 µs
(82.05%)
nested_routers/nested_routers📈 view plot
🚷 view threshold
1,041.50 µs
(+7.94%)Baseline: 964.92 µs
1,251.91 µs
(83.19%)
single_root_route/single_root_route📈 view plot
🚷 view threshold
987.86 µs
(+6.37%)Baseline: 928.66 µs
1,214.19 µs
(81.36%)
single_root_route_burst/single_root_route_burst📈 view plot
🚷 view threshold
16,654.00 µs
(-1.31%)Baseline: 16,874.30 µs
21,617.51 µs
(77.04%)
🐰 View full continuous benchmarking report in Bencher

@ElijahAhianyo ElijahAhianyo changed the title elijah/cot cli proxy/stack 3 proxy feat(cli): proxy unrecognized commands to the project binary Sep 3, 2026
@ElijahAhianyo
ElijahAhianyo marked this pull request as ready for review September 3, 2026 05:12
@ElijahAhianyo
ElijahAhianyo force-pushed the elijah/cot-cli-proxy/stack-3-proxy branch 2 times, most recently from c2d07b3 to f551256 Compare September 3, 2026 05:24
@ElijahAhianyo
ElijahAhianyo force-pushed the elijah/cot-cli-proxy/stack-3-proxy branch from 43e4874 to c06cfb8 Compare September 3, 2026 18:05
@ElijahAhianyo
ElijahAhianyo requested review from a team and removed request for a team September 3, 2026 18:33
@ElijahAhianyo
ElijahAhianyo force-pushed the elijah/cot-cli-proxy/stack-3-proxy branch from c06cfb8 to 9442953 Compare September 7, 2026 16:07

@m4tx m4tx left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, but make sure the tests pass and have a look at my comment.

Comment thread cot-cli/tests/cli.rs Outdated
Base automatically changed from elijah/cot-cli-proxy/stack-2-discovery to master September 10, 2026 19:41
@ElijahAhianyo
ElijahAhianyo force-pushed the elijah/cot-cli-proxy/stack-3-proxy branch 2 times, most recently from 015d4bf to 359eb86 Compare September 11, 2026 00:22
@github-actions github-actions Bot added A-deps Area: Dependencies C-lib Crate: cot (main library crate) labels Sep 11, 2026
@ElijahAhianyo
ElijahAhianyo force-pushed the elijah/cot-cli-proxy/stack-3-proxy branch from 359eb86 to d063ae8 Compare September 11, 2026 00:46
@codecov

codecov Bot commented Sep 11, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 81.93277% with 129 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
cot-cli/src/test_harness.rs 71.32% 93 Missing and 22 partials ⚠️
cot-cli/src/main.rs 80.28% 13 Missing and 1 partial ⚠️
Flag Coverage Δ
rust 90.04% <81.93%> (-0.25%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
cot-cli/src/args.rs 100.00% <100.00%> (ø)
cot-cli/src/handlers.rs 98.41% <100.00%> (+2.48%) ⬆️
cot-cli/src/project.rs 96.96% <ø> (ø)
cot-cli/src/main.rs 84.09% <80.28%> (-15.91%) ⬇️
cot-cli/src/test_harness.rs 71.32% <71.32%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ElijahAhianyo
ElijahAhianyo force-pushed the elijah/cot-cli-proxy/stack-3-proxy branch from 104e5fb to e509eda Compare September 14, 2026 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-deps Area: Dependencies C-cli Crate: cot-cli (issues and Pull Requests related to Cot CLI) C-lib Crate: cot (main library crate)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants