Why use many word when few do?
Claude Code skills that prevent bad work before it ships. 40 skills. Caveman voice. Finished product.
Same /review. Same patch. Watch the difference.
# patch /review Analyzing your changes... I'll review the changes in auth/middleware.ts and provide comprehensive feedback on the implementation. ## Summary This pull request modifies the authentication middleware to add token expiry validation. The changes look generally good, but I have several observations and recommendations that I'd like to share with you. ## Detailed Analysis ### Finding 1: Token Expiry Check (Medium) In auth/middleware.ts:47, the token expiry comparison uses a strict less-than operator (<) when it should use less-than-or-equal-to (<=). This means tokens that expire at exactly the current timestamp will incorrectly be treated as valid. Recommendation: Change the comparison operator from < to <= to ensure tokens are invalidated at exactly their expiry time. ### Finding 2: Missing Error Context (Low) The error response at line 52 returns a generic 401 without diagnostic information. Consider adding a "reason" field to the error response. ## Overall Assessment The changes demonstrate a solid understanding of the authentication flow. With the operator fix applied, this should be ready to merge.
# patch /review
## Findings: 2
1. auth/middleware.ts:47 — token expiry check
use < not <=. Token at exact expiry time
passes validation. Fix:
- if (token.exp < now) {
+ if (token.exp <= now) {
2. auth/middleware.ts:52 — 401 response
missing reason field. Add:
{ error: "unauthorized", reason: "token_expired" }
Status: Fix #1, merge.
Six judgment-layer skills prevent bad work before it ships. 34 more under /skills.
/help inside Claude Code to list them. No flag needed.
see all →
Keep all gstack functionality. Upgrade it. Ship it finished.
Two grunt. That all.
curl -fsSL https://cavestack.jerkyjesse.com/install | sh
/help or /office-hours. Done.claude # caveman hooks fire on SessionStart
Already have gstack/cavestack? /cavestack-upgrade handles the migration.
Three intensity. Pick grunt level.