Skip to content

feat(ls): list every rule over qbx/rules - #14

Merged
ChatDisabled merged 1 commit into
Qbox-project:mainfrom
loaf-scripts:feat/rule-suggestions
Oct 1, 2026
Merged

ChatDisabled merged 1 commit into
Qbox-project:mainfrom
loaf-scripts:feat/rule-suggestions

Conversation

@loaf-scripts

@loaf-scripts loaf-scripts commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Changes

Adds a qbx/rules request that returns every rule with its code, category, default level (off for rules that are off by default), fixable and summary. It reads rules::RULES, so the list always matches the server that answers it.

The VS Code extension uses it to suggest rule names in qbxLua.diagnostics.rules and to flag names the server doesn't know (Qbox-project/qbx-editor#6). The request is added to the table in crates/qbx_lua_ls/docs/protocol.md.

The branch is based on the current main and doesn't depend on any other change.

Checks

  • cargo fmt --all -- --check
  • cargo clippy -p qbx_lua_ls --all-targets --locked -- -D warnings
  • cargo test -p qbx_lua_ls --locked, including the new lists_every_rule_with_its_default_level
  • Used from the editor branch in a real VS Code instance (see the editor PR)

🤖 Generated with Claude Code

@ChatDisabled
ChatDisabled merged commit 54c9a8b into Qbox-project:main Oct 1, 2026
2 checks passed
@loaf-scripts
loaf-scripts deleted the feat/rule-suggestions branch October 1, 2026 07:37
@loaf-scripts
loaf-scripts restored the feat/rule-suggestions branch October 1, 2026 07:37
@loaf-scripts
loaf-scripts deleted the feat/rule-suggestions branch October 1, 2026 07:38
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