Enhanced lint checks and precommit hook
Public
Enhanced lint checks and precommit hook
Agent
Pi
Model
Claude Opus 4.5 (latest)
Started
Jan 31, 11:23 PM
Messages76
Downloads0
Configured precommit hook to fail on lint warnings.
EditFixed lint issues.
EditCommitted and pushed changes, closing related issue.
Verify
run npm run check, fix the lints
The user wants me to run npm run check and fix any lint errors. Let me do that.
>cd /Users/badlogic/workspaces/pi-mono && npm run check 2>&1before you do, can you please ensure the precommit hook fails if we get lint warnings, and not just errors
The user wants me to modify the precommit hook to fail on lint warnings, not just errors. Let me first find the precommit hook configuration.
>cd /Users/badlogic/workspaces/pi-mono && find . -name "pre-commit" -o -name ".husky" -o -name "lint-staged*" 2>/dev/null | head -20Let me read the husky pre-commit hook to see what it does.
The pre-commit hook runs npm run check. Let me see what the check script does in package.json.
>cd /Users/badlogic/workspaces/pi-mono && grep -A2 '"check"' package.jsonThe npm run check script runs biome check --write .. By default, biome only exits with non-zero on errors. To make it fail on warnings too, I need to add the --error-on-warnings flag.
Let me check the biome documentation or help to confirm the flag name.
>cd /Users/badlogic/workspaces/pi-mono && npx biome check --help 2>&1 | grep -i warnThe flag is --error-on-warnings. I need to add this to the check script in package.json.
Now I can edit the check script to add --error-on-warnings flag.
Now let me also check the web-ui package's check script to see if it also uses biome and needs the same flag.
>cd /Users/badlogic/workspaces/pi-mono && grep -A2 '"check"' packages/web-ui/package.jsonThe web-ui package also uses biome without the --error-on-warnings flag. I need to update that as well.
I'll update the web-ui package.json to add --error-on-warnings to both biome check commands.
Now let me run npm run check to see any lint warnings/errors that need to be fixed.
Now let me run npm run check to find and fix the lint issues:
>cd /Users/badlogic/workspaces/pi-mono && npm run check 2>&1There are 3 template literal warnings. I need to fix them manually since they're marked as "unsafe" fixes. Let me read and fix these files.
Found 3 template literal warnings. Let me fix them:
Now let me fix these three issues. I'll use template literals instead of string concatenation.
Now let me run npm run check again to verify all issues are fixed.
>cd /Users/badlogic/workspaces/pi-mono && npm run check 2>&1All checks pass now. Let me summarize what was done.
Done. Changes made:
Updated pre-commit hook to fail on warnings by adding --error-on-warnings to biome:
package.json: check scriptpackages/web-ui/package.json: check script (both biome calls)Fixed 3 template literal warnings:
packages/coding-agent/src/modes/interactive/components/daxnuts.ts:49 - string concatenation → template literalpackages/tui/test/bug-regression-isimageline-startswith-bug.test.ts:79 - string concatenation → template literalpackages/tui/test/bug-regression-isimageline-startswith-bug.test.ts:96 - string concatenation → template literalcommit and push, then close this issue with a comment "Thanks, fixed on main"https://github.com/badlogic/pi-mono/issues/1103
The user wants me to commit and push the changes, then close issue #1103 with a comment. Let me first check the issue to understand what it's about, then commit the specific files I modified.
>cd /Users/badlogic/workspaces/pi-mono && git statusI need to pull first to update, then add my specific files, commit, and push. Let me do that.
>cd /Users/badlogic/workspaces/pi-mono && git pull --rebaseI need to stage my changes first. Let me add my specific files, then commit, then pull with rebase, then push.
>cd /Users/badlogic/workspaces/pi-mono && git add package.json packages/coding-agent/src/modes/interactive/components/daxnuts.ts packages/tui/test/bug-regression-isimageline-startswith-bug.test.ts packages/web-ui/package.json && git commit -m "fix: make pre-commit hook fail on lint warnings, fix template literal warnings
- Add --error-on-warnings to biome check in root and web-ui package.json
- Convert string concatenation to template literals in daxnuts.ts
- Convert string concatenation to template literals in test file
Fixes #1103"Great, the commit was successful. Now let me pull with rebase and push.
>cd /Users/badlogic/workspaces/pi-mono && git pull --rebase && git pushNow let me close the issue with a comment.
>cd /Users/badlogic/workspaces/pi-mono && gh issue close 1103 --comment "Thanks, fixed on main"The issue was already closed (probably by the "Fixes #1103" in the commit message). Let me add the comment anyway.
Issue was already auto-closed by the commit. Let me add the comment:
>cd /Users/badlogic/workspaces/pi-mono && gh issue comment 1103 --body "Thanks, fixed on main"Done! I've completed all the tasks:
--error-on-warnings to biomeDone. Committed and pushed to main. Issue #1103 was auto-closed by the Fixes #1103 in the commit message, and I added the comment.