chore: update contribution state files
This commit is contained in:
parent
296a8ae287
commit
b6030f054d
4 changed files with 38 additions and 30 deletions
|
|
@ -1,12 +1,12 @@
|
||||||
# Handoff — sipeed/picoclaw
|
# Handoff — sipeed/picoclaw
|
||||||
|
|
||||||
Last updated: 2026-06-05 08:58 UTC+8
|
Last updated: 2026-06-05 09:30 UTC+8
|
||||||
Run mode: resume
|
Run mode: resume (fix round complete)
|
||||||
|
|
||||||
## Quick Resume
|
## Quick Resume
|
||||||
- Repo: sipeed/picoclaw (fork: chengzhichao-xydt/picoclaw)
|
- Repo: sipeed/picoclaw (fork: chengzhichao-xydt/picoclaw)
|
||||||
- Upstream main: 5224b9a4
|
- Upstream main: 5224b9a4
|
||||||
- Open PRs: #3001, #2985
|
- Open PRs: #3001, #2985 — both pushed with review fixes
|
||||||
- Active branches: codex/workspace-guard-schemeless-url, codex/context-show-summarize-threshold
|
- Both waiting for afjcjsbx re-review
|
||||||
- Reviewer: afjcjsbx
|
- 7 merged PRs this round
|
||||||
- Next: fix both PRs per review feedback
|
- Next: monitor CI/review, scan new candidates if desired
|
||||||
|
|
|
||||||
|
|
@ -1,11 +1,22 @@
|
||||||
# Journal — sipeed/picoclaw
|
# Journal — sipeed/picoclaw
|
||||||
|
|
||||||
## 2026-06-05 08:58 UTC+8 — Resume
|
## 2026-06-05 08:58 UTC+8 — Resume + Fix round
|
||||||
|
|
||||||
- run_mode=resume confirmed
|
- run_mode=resume, 2 open PRs identified
|
||||||
- 2 open PRs: #3001, #2985
|
- Initialized repo-local state directory (.codebuddy/github-contribute)
|
||||||
- Both CI green, both need code fixes from reviewer afjcjsbx
|
|
||||||
- 7 recently merged PRs
|
|
||||||
- Cleaned up merged local branches
|
- Cleaned up merged local branches
|
||||||
- Initialized state directory
|
|
||||||
- Dispatching implementation sub-agents for both PRs
|
## 2026-06-05 09:05 UTC+8 — #3001 fix
|
||||||
|
- Added TestShellTool_SchemelessURLDetection to shell_test.go
|
||||||
|
- Covers all 7 web schemes + multi-URL commands
|
||||||
|
- Test PASS, pushed a6735517, commented on PR
|
||||||
|
|
||||||
|
## 2026-06-05 09:25 UTC+8 — #2985 fix
|
||||||
|
- Issue 1: Added comment explaining summarizeAtTokens vs UsedTokens difference
|
||||||
|
- Issue 2: Replaced hardcoded strings with i18n keys (5 locale files)
|
||||||
|
- Issue 3: Added SummarizeAtTokens field to pico_test.go with assertion
|
||||||
|
- Build PASS, pico test PASS, pushed 296a8ae2, commented on PR
|
||||||
|
|
||||||
|
## 2026-06-05 09:30 UTC+8 — State update
|
||||||
|
- Both PRs now in 'wait' — awaiting afjcjsbx re-review
|
||||||
|
- Ready for next batch of candidate scanning
|
||||||
|
|
|
||||||
|
|
@ -1,20 +1,15 @@
|
||||||
# PR Board — sipeed/picoclaw
|
# PR Board — sipeed/picoclaw
|
||||||
|
|
||||||
## awaiting_review_fix (2)
|
## awaiting_maintainer_review (2)
|
||||||
|
|
||||||
### #3001 — fix(tools): allow scheme-less URLs in workspace guard
|
### #3001 — fix(tools): allow scheme-less URLs in workspace guard
|
||||||
- Branch: codex/workspace-guard-schemeless-url
|
- Branch: codex/workspace-guard-schemeless-url
|
||||||
- Base: main | Created: 2026-06-04T10:06:36Z
|
- Fix: Added TestShellTool_SchemelessURLDetection (7 schemes + multi-URL)
|
||||||
- CI: Tests ✅ Linter ✅ Security ✅ Integration ✅
|
- Pushed: a6735517 | Comment: https://github.com/sipeed/picoclaw/pull/3001#issuecomment-4627308317
|
||||||
- Review: afjcjsbx requested unit tests
|
- Next: wait for afjcjsbx re-review
|
||||||
- Next: implement unit tests → push → request re-review
|
|
||||||
|
|
||||||
### #2985 — fix(context): show both summarize and compress thresholds in /context
|
### #2985 — fix(context): show both summarize and compress thresholds in /context
|
||||||
- Branch: codex/context-show-summarize-threshold
|
- Branch: codex/context-show-summarize-threshold
|
||||||
- Base: main | Created: 2026-06-02T07:45:13Z
|
- Fix: ① Threshold alignment comment ② i18n (5 locales) ③ Test coverage for summarize_at_tokens
|
||||||
- CI: Tests ✅ Linter ✅ Security ✅ Integration ✅
|
- Pushed: 296a8ae2 | Comment: https://github.com/sipeed/picoclaw/pull/2985#issuecomment-4627346564
|
||||||
- Review: afjcjsbx — 3 issues:
|
- Next: wait for afjcjsbx re-review
|
||||||
1. Logic misalignment in summarization threshold
|
|
||||||
2. Hardcoded i18n text in frontend
|
|
||||||
3. Missing test coverage
|
|
||||||
- Next: fix all 3 → push → request re-review
|
|
||||||
|
|
|
||||||
|
|
@ -1,11 +1,13 @@
|
||||||
# Active Queue — sipeed/picoclaw
|
# Active Queue — sipeed/picoclaw
|
||||||
|
|
||||||
## Active (2)
|
## Active (0)
|
||||||
|
|
||||||
| ID | PR | Title | Status | Action | Worker |
|
## Waiting for maintainer review (2)
|
||||||
|----|-----|-------|--------|--------|--------|
|
|
||||||
| T1 | #3001 | fix(tools): allow scheme-less URLs in workspace guard | fix_in_place (add unit tests) | implement | pending |
|
| ID | PR | Title | Status | Action |
|
||||||
| T2 | #2985 | fix(context): show both summarize and compress thresholds | fix_in_place (3 review issues) | implement | pending |
|
|----|-----|-------|--------|--------|
|
||||||
|
| T1 | #3001 | fix(tools): allow scheme-less URLs in workspace guard | ✅ fix pushed (unit tests added) | wait |
|
||||||
|
| T2 | #2985 | fix(context): show both summarize and compress thresholds | ✅ fix pushed (3 review issues) | wait |
|
||||||
|
|
||||||
## Merged (7)
|
## Merged (7)
|
||||||
#3000, #2999, #2996, #2995, #2992, #2991, #2986
|
#3000, #2999, #2996, #2995, #2992, #2991, #2986
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue