pikers
/
piker
4
1
Fork 0
Code Issues 20 Pull Requests 11 Packages Projects Releases Wiki Activity

claudy_writes_ruff_conf: a skill for linting to our (my) "wonky code style" #88

Open
goodboy wants to merge 3 commits from claudy_writes_ruff_conf into moar_ai_skillz
pull from: claudy_writes_ruff_conf
merge into: pikers:moar_ai_skillz
Conversation 0 Commits 3 Files Changed 3
goodboy commented 2026-03-12 01:08:36 +00:00
Owner

Pertains to further extending claude skills as proposed in #79 in this case trying to get a base ruff.conf + surrounding tooling to encourage a common style throughout (at least to start) all the py codez 🙏

Also see https://www.pikers.dev/pikers/piker/issues/87#issuecomment-1431 for my incremental updates from the prompt-sesh where all this was vibed.

What to do

  • test and doc the new /lint skill-cmd.
  • ensure we have a sane setup even on nixos dev envs.
    • originally there were questions about “which ruff” should be used.
  • address the auto-fixable and non cases_,

    • 274 auto-fixable with ruff check –fix.

    • D205 (blank line between summary + description) is not auto-fixable, so those 411 violations need manual attention or a scripted fix.

Pertains to further extending `claude` skills as proposed in https://pikers.dev/pikers/piker/issues/79 in this case trying to get a base `ruff.conf` + surrounding tooling to encourage a common style throughout (at least to start) all the py codez 🙏 Also see https://www.pikers.dev/pikers/piker/issues/87#issuecomment-1431 for my incremental updates from the prompt-sesh where all this was vibed. --- #### What to do - [ ] test and doc the new `/lint` skill-cmd. * prolly just use the [comment here verbatim?](https://www.pikers.dev/pikers/piker/issues/87#issuecomment-1431) - [ ] ensure we have a sane setup even on nixos dev envs. * originally there were questions about "which `ruff`" should be used. - [ ] address [the *auto-fixable* and non cases](https://www.pikers.dev/pikers/piker/issues/87#issue-141)_, * > 274 auto-fixable with ruff check --fix. * > D205 (blank line between summary + description) is not auto-fixable, so those 411 violations need manual attention or a scripted fix.
goodboy added 3 commits 2026-03-12 01:08:36 +00:00
cec6d19156 Add pydocstyle and `W` rules to `ruff.toml` 
Enable `D2xx` formatting rules to enforce piker's `'''`
multiline docstring convention: summary on second line
(D213), closing quotes on separate line (D209), blank
line between summary and description (D205).

Deats,
- select `D2` (whitespace/formatting) rule group plus
  cherry-picked `D402`/`D403`/`D419` content checks.
- ignore `D200` (allow multiline for short docstrings),
  `D203` (use `D211` instead), `D212` (use `D213`).
- skip `D1xx` (missing-docstring) to avoid noise and
  `D3xx` since `D300` conflicts with `'''` convention.
- add `W` rules for trailing-whitespace detection.

Also,
- enable `docstring-code-format` with 67-char width
  for code examples inside docstrings.

(this patch was generated in some part by [`claude-code`][claude-code-gh])
[claude-code-gh]: https://github.com/anthropics/claude-code
e5a91faac7 Add `/lint` skill with `scripts/check.py` 
Add a user-invocable `/lint` skill that wraps
`ruff check` with staged-file defaults and piker's
`ruff.toml` config via a PEP 723 script.

The `ruff.toml` (from prior commit) now enforces
piker's `'''` multiline docstring convention with
`D2xx` formatting rules: summary on second line
(D213), closing quotes on separate line (D209), blank
line between summary and description (D205), plus
`W` trailing-whitespace and cherry-picked `D4xx`
content checks.

Deats,
- `scripts/check.py`: pure-stdlib wrapper that
  defaults to staged `.py` files via
  `git diff --cached`, with `--all`, `--fix`,
  `--diff`, and `--stats` flags.
- calls `ruff` from `$PATH` (works with nixpkgs,
  nvim-bundled, or pip-installed).
- prints clear install guidance if `ruff` not found.
- `SKILL.md`: documents usage, common violations
  (D213, D205, D204), and fix patterns for
  converting to piker's docstring style.

(this patch was generated in some part by [`claude-code`][claude-code-gh])
[claude-code-gh]: https://github.com/anthropics/claude-code
goodboy force-pushed claudy_writes_ruff_conf from e5a91faac7 to fd8d81becf 2026-03-17 22:04:43 +00:00 Compare
goodboy force-pushed claudy_writes_ruff_conf from fd8d81becf to f55c4fce96 2026-03-27 16:05:35 +00:00 Compare
goodboy changed title from claudy_writes_ruff_conf: a skill for linting to our (my) "wonky code style" to claudy_writes_ruff_conf: a skill for linting to our (my) "wonky code style" 2026-03-27 16:07:47 +00:00
goodboy changed title from claudy_writes_ruff_conf: a skill for linting to our (my) "wonky code style" to claudy_writes_ruff_conf: a skill for linting to our (my) "wonky code style" 2026-03-27 16:10:31 +00:00
goodboy changed target branch from main to moar_ai_skillz 2026-03-27 16:10:31 +00:00
goodboy force-pushed claudy_writes_ruff_conf from f55c4fce96 to bb2ea50f42 2026-04-07 19:28:36 +00:00 Compare
This pull request can be merged automatically.
You are not authorized to merge this pull request.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b claudy_writes_ruff_conf moar_ai_skillz
git pull origin claudy_writes_ruff_conf

Step 2:

Merge the changes and update on Gitea.
git checkout moar_ai_skillz
git merge --no-ff claudy_writes_ruff_conf
git push origin moar_ai_skillz
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: pikers/piker#88
There is no content yet.
Delete Branch "claudy_writes_ruff_conf"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?