Skip to content

Implement codesmell as a new linter - #9

Open
hungpham10 wants to merge 10 commits into
mainfrom
feature/codesmell-mvp-as-a-lint-tool-to-help-llm-to-write-maintainable-code
Open

Implement codesmell as a new linter#9
hungpham10 wants to merge 10 commits into
mainfrom
feature/codesmell-mvp-as-a-lint-tool-to-help-llm-to-write-maintainable-code

Conversation

@hungpham10

Copy link
Copy Markdown
Owner

No description provided.

@coderabbitai

coderabbitai Bot commented Aug 16, 2026

Copy link
Copy Markdown

Review Change Stack

Important

Review available on request

  • 🔍 Trigger review

Reviews should be triggered manually for repositories with fewer than 10 stars. Select Trigger review above or comment @coderabbitai review to review the latest changes. For a full review, comment @coderabbitai full review.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 1ba6554f-b665-41ab-ab69-ff1a0dcde7ab

✨ 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 feature/codesmell-mvp-as-a-lint-tool-to-help-llm-to-write-maintainable-code

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.

@codecov-commenter

codecov-commenter commented Aug 16, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 67.87709% with 230 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.07%. Comparing base (eedc0c4) to head (4477abd).

Files with missing lines Patch % Lines
crates/codesmell/src/main.rs 0.00% 112 Missing ⚠️
crates/codesmell/src/guide.rs 0.00% 59 Missing ⚠️
crates/codesmell/src/rules.rs 88.31% 34 Missing ⚠️
crates/codesmell/src/policy.rs 89.24% 17 Missing ⚠️
crates/codesmell/src/engine.rs 88.73% 8 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #9      +/-   ##
==========================================
- Coverage   74.34%   74.07%   -0.28%     
==========================================
  Files          67       74       +7     
  Lines       13989    14705     +716     
==========================================
+ Hits        10400    10892     +492     
- Misses       3589     3813     +224     

☔ View full report in Codecov by Harness.
📢 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.

@codspeed-hq

codspeed-hq Bot commented Aug 16, 2026

Copy link
Copy Markdown

Merging this PR will degrade performance by 70.49%

⚠️ Different runtime environments detected

Some benchmarks with significant performance changes were compared across different runtime environments,
which may affect the accuracy of the results.

Open the report in CodSpeed to investigate

❌ 24 regressed benchmarks
✅ 4 untouched benchmarks

Warning

Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Benchmark BASE HEAD Efficiency
open+ingest 85.8 ms 1,007.3 ms -91.48%
ingest 86.1 ms 1,007.2 ms -91.46%
ingest 76.9 ms 896.5 ms -91.42%
open+ingest 77.4 ms 896.4 ms -91.37%
ingest 22.7 ms 227.9 ms -90.03%
open+ingest 22.9 ms 228 ms -89.94%
ingest 13.3 ms 113.6 ms -88.28%
open+ingest 13.4 ms 113.5 ms -88.23%
open+ingest 248.7 ms 1,169.1 ms -78.73%
open+ingest 226.3 ms 1,047.1 ms -78.39%
open+ingest 68.5 ms 273.3 ms -74.94%
open+ingest 35.8 ms 136.5 ms -73.81%
open+ingest 214 ms 419.4 ms -48.98%
open+ingest 109.6 ms 211.9 ms -48.3%
open+ingest 1.1 s 2 s -46.86%
open+ingest 960.5 ms 1,801.5 ms -46.69%
sample 142.7 ms 193.9 ms -26.39%
sample 142.8 ms 193.8 ms -26.33%
sample 60.2 ms 81.1 ms -25.74%
sample 60.2 ms 81.1 ms -25.74%
... ... ... ... ...

ℹ️ Only the first 20 benchmarks are displayed. Go to the app to view all benchmarks.

Tip

Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.


Comparing feature/codesmell-mvp-as-a-lint-tool-to-help-llm-to-write-maintainable-code (a27384c) with main (eedc0c4)

Open in CodSpeed

@hungpham10
hungpham10 force-pushed the feature/codesmell-mvp-as-a-lint-tool-to-help-llm-to-write-maintainable-code branch from 9d45800 to 4477abd Compare August 16, 2026 15:53
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.

2 participants