Skip to content

Fix: improve CLI help, Fibonacci example, LSP completions, and benchmark metrics (#9 #13 #14 #20) - #54

Merged
Hesara2003 merged 5 commits into
ieeecsopen:mainfrom
IT24100284:Yetmi
Sep 16, 2026
Merged

Hesara2003 merged 5 commits into
ieeecsopen:mainfrom
IT24100284:Yetmi

Conversation

@IT24100284

@IT24100284 IT24100284 commented Sep 13, 2026 •

Copy link
Copy Markdown
Contributor

Description

This PR addresses several remaining tooling, documentation, editor-experience, and benchmarking improvements in the NOVA project.

Included Fixes

  • Improved the CLI help output to make available commands and options clearer.
  • Updated the Fibonacci example for better clarity and usability.
  • Improved LSP completions to provide a better developer experience when working with NOVA.
  • Enhanced benchmark metrics to provide more useful and consistent performance measurements.

Related Issues

Closes #9
Closes #13
Closes #14
Closes #20

Summary

These changes improve the overall developer experience and make NOVA's tooling, examples, editor support, and performance benchmarking more consistent and user-friendly.

Summary by CodeRabbit

  • New Features

    • Improved NOVA CLI help with clearer descriptions and practical examples for development, builds, and checks.
    • Enhanced editor completions with type-specific details and documentation for standard library types, keywords, and capabilities.
  • Bug Fixes

    • Updated completion metadata so users receive more informative guidance while editing NOVA code.
  • Tests

    • Added coverage to verify CLI examples and rich completion information remain available.

@coderabbitai

coderabbitai Bot commented Sep 13, 2026 •

Copy link
Copy Markdown

Review Change StackReview Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 3d72e231-1427-49bd-9333-e02bc8caadf7

📥 Commits

Reviewing files that changed from the base of the PR and between 8cd2e03 and 5499d59.

📒 Files selected for processing (4)
  • compiler/nova_compiler/cli.py
  • compiler/nova_compiler/lsp_server.py
  • editors/vscode/test/lsp_smoke.py
  • tests/toolchain/run.py

📝 Walkthrough

Walkthrough

The CLI now shows command examples in help output. LSP completion items now provide type-specific detail and documentation. Tests verify the CLI examples and enriched completion metadata.

Changes

Tooling metadata improvements

Layer / File(s) Summary
CLI help examples and regression coverage
compiler/nova_compiler/cli.py, tests/toolchain/run.py
The CLI help uses raw descriptions and lists dev, build, and check examples. Subcommand help text includes command-specific examples. A regression test verifies the examples and successful execution.
LSP completion metadata and smoke validation
compiler/nova_compiler/lsp_server.py, editors/vscode/test/lsp_smoke.py
Standard library types, keywords, and capabilities now provide completion detail and documentation. The smoke test requires metadata for the Option completion item.

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Feature

Suggested reviewers: tharupinsara

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit 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.

@Hesara2003
Hesara2003 merged commit bd836a2 into ieeecsopen:main Sep 16, 2026
1 check was pending
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

4 participants