replit/token-gated-economy
Archived
2
0
Fork 0
Code Issues 15 Pull Requests Actions Packages Projects Releases Wiki Activity

[claude] fix testkit rate-limit false failures (#25) #54

Merged
Rockachopa merged 1 commits from claude/issue-25 into main 2026-03-23 14:51:32 +00:00
Conversation 0 Commits 1 Files Changed 2 +104 -42
claude commented 2026-03-23 01:05:22 +00:00
Collaborator
Copy Link

Fixes #25

Changes

  1. Reordered testkit: moved test 8c (demo param validation) before tests 7 and 9 so rate-limit quota is not consumed before validation is checked — eliminates both false failures (tests 7 and 8c)
  2. macOS compatibility: replaced all 19 head -n-1 calls with sed $d (GNU head negative-offset syntax is not available on macOS)
  3. Archived report: saved the Claude Opus 4.6 testkit report to reports/timmy-testkit-claude-opus.md

Test plan

  • Shell script syntax validated (bash -n)
  • No TypeScript changes — only shell script and markdown
  • Rebased on latest main (no conflicts)
Fixes #25 ## Changes 1. **Reordered testkit**: moved test 8c (demo param validation) before tests 7 and 9 so rate-limit quota is not consumed before validation is checked — eliminates both false failures (tests 7 and 8c) 2. **macOS compatibility**: replaced all 19 `head -n-1` calls with `sed $d` (GNU head negative-offset syntax is not available on macOS) 3. **Archived report**: saved the Claude Opus 4.6 testkit report to `reports/timmy-testkit-claude-opus.md` ## Test plan - Shell script syntax validated (`bash -n`) - No TypeScript changes — only shell script and markdown - Rebased on latest main (no conflicts)
claude added 1 commit 2026-03-23 01:05:23 +00:00
fix: reorder testkit to prevent rate-limit false failures
Some checks failed
CI / Typecheck & Lint (pull_request) Failing after 1s
Details
82e198cbc4 
Move test 8c (demo missing param validation) before tests 7 and 9 so
the demo rate-limit quota is not consumed before param validation is
checked. Replace `head -n-1` with `sed '$d'` throughout for macOS
compatibility. Archive Claude Opus 4.6 testkit report.

Fixes #25

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
claude force-pushed claude/issue-25 from 82e198cbc4 to 52baa0fe92 2026-03-23 02:07:49 +00:00 Compare
Rockachopa merged commit 72c2c8dee7 into main 2026-03-23 14:51:32 +00:00
This repo is archived. You cannot comment on pull requests.
Reviewers
No Reviewers
Labels
Clear labels
api
Backend API work
 backend
API server changes
 enhancement
Feature enhancement
 epic
Epic / parent issue
 epic
Top-level epic issue
 frontend
Three.js Workshop frontend
 frontend
Three.js / Tower frontend changes
 hermes
Assigned to Hermes
 infra
Infrastructure / deployment
 integration
Cross-system integration
 kimi
Assigned to Kimi agent
 replit
Assigned to Replit agent
 timmy
Timmy autonomous agent work — economics, identity, relay, proactive engagement
 world
3D world / Tower issues
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
Rockachopa claude gemini google hermes manus replit
No Assignees
1 Participants
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: replit/token-gated-economy#54
Delete Branch "claude/issue-25"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?