Skip to content

Improved path comp#17

Merged
Arthurdw merged 15 commits intomainfrom
improved-path-comp
Jan 9, 2026
Merged

Improved path comp#17
Arthurdw merged 15 commits intomainfrom
improved-path-comp

Conversation

@Arthurdw
Copy link
Owner

@Arthurdw Arthurdw commented Jan 8, 2026

Summary by CodeRabbit

  • New Features

    • Automatic OpenAPI support for primitive and tuple path parameters; editor support for completions, go-to, references, rename, and hover.
  • Documentation

    • New "Path Parameters" guidance with examples; README, examples, and rustdoc updated.
  • Bug Fixes / Diagnostics

    • Actionable warnings for undocumented, mismatched, or missing path parameters.
  • Tests

    • Extensive test coverage for parsing, generation, diagnostics, editor actions, and rename/validation flows.
  • Chores

    • Pre-commit hooks now install frontend deps only when needed.

✏️ Tip: You can customize this high-level summary in your review settings.

@Arthurdw Arthurdw self-assigned this Jan 8, 2026
@Arthurdw Arthurdw added the enhancement New feature or request label Jan 8, 2026
@coderabbitai

This comment was marked as resolved.

@codecov
Copy link

codecov bot commented Jan 8, 2026

Codecov Report

❌ Patch coverage is 92.96830% with 122 lines in your changes missing coverage. Please review.
✅ Project coverage is 91.89%. Comparing base (4f68f93) to head (c00fe7d).
⚠️ Report is 17 commits behind head on main.

Files with missing lines Patch % Lines
rovo-lsp/src/handlers.rs 87.24% 80 Missing ⚠️
rovo-lsp/src/completion.rs 95.76% 15 Missing ⚠️
rovo-macros/src/parser/tokens.rs 96.09% 13 Missing ⚠️
rovo-lsp/src/diagnostics.rs 95.48% 6 Missing ⚠️
rovo-lsp/src/parser.rs 95.52% 3 Missing ⚠️
rovo-lsp/src/code_actions.rs 98.34% 2 Missing ⚠️
rovo-macros/src/lib.rs 96.61% 2 Missing ⚠️
rovo-macros/src/parser/mod.rs 97.56% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #17      +/-   ##
==========================================
+ Coverage   91.39%   91.89%   +0.50%     
==========================================
  Files          17       17              
  Lines        3776     5505    +1729     
==========================================
+ Hits         3451     5059    +1608     
- Misses        325      446     +121     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

coderabbitai[bot]

This comment was marked as resolved.

coderabbitai[bot]

This comment was marked as resolved.

coderabbitai[bot]

This comment was marked as resolved.

coderabbitai[bot]

This comment was marked as resolved.

- Add edge case tests for tokens.rs (empty tokens, nested generics,
  duplicate bindings, malformed input)
- Add coverage tests for completion.rs (section context detection,
  response/example completions, security schemes)
- Add validation tests for diagnostics.rs (example validation,
  multiline examples, function visibility patterns)
- Fix pre-commit hook operator precedence for VSCode extension
  npm install check
coderabbitai[bot]

This comment was marked as resolved.

coderabbitai[bot]

This comment was marked as resolved.

coderabbitai[bot]

This comment was marked as resolved.

@Arthurdw Arthurdw added the release:minor Minor version bump label Jan 9, 2026
@Arthurdw Arthurdw merged commit d65f459 into main Jan 9, 2026
24 checks passed
@Arthurdw Arthurdw deleted the improved-path-comp branch January 9, 2026 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request release:minor Minor version bump

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant