docs: recompute runner script and review reports are untracked, rollout steps unreferenced
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 65/100
- Loại issue
- Tài liệu
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- shell, typescript
- Lĩnh vực
- devops, documentation
Hướng nghiên cứu
Start with the usage header in apps/web/app/routes/api.admin.recompute.ts and inspect scripts/run-recompute.sh, then review docs/deployment-guide.md, docs/UPDATE_SUMMARY.md, and the two plans/reports files. Use git status --porcelain and the documented grep check to verify the files are handled and the script is referenced. Done means the script and reports are committed or deliberately deleted, the update summary is resolved, and the runbook matches the endpoint’s live behavior.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Summary
The bulk-recompute runbook tooling and two code-review reports exist only in the working tree. No commit contains them, and nothing under docs/ references the runner script, so the bulk recompute rollout cannot be reproduced from a clean checkout.
Location
- Doc:
docs/deployment-guide.md(runbook section, currently silent on bulk recompute) anddocs/UPDATE_SUMMARY.md - Source of truth:
apps/web/app/routes/api.admin.recompute.ts:1-12(the endpoint's own usage header) andscripts/run-recompute.sh
Problem
git status --porcelain reports these paths as untracked:
?? docs/UPDATE_SUMMARY.md
?? plans/reports/code-review-260305-skill-references-scripts.md
?? plans/reports/code-reviewer-260306-1531-admin-recompute-endpoint.md
?? scripts/run-recompute.sh
scripts/run-recompute.shis the operational driver for the recompute endpoint: it defaultsAPI_URLtohttps://skillx.sh/api/admin/recompute, requiresADMIN_SECRET, takes a batch size argument, and supports resume by re-running.grep -rn 'run-recompute' --include=*.md --include=*.json .excludingnode_modulesreturns no matches, so the script is unreferenced and a maintainer has no documented way to find it.docs/UPDATE_SUMMARY.mddescribes the Mar 5 documentation pass and overlapsdocs/project-roadmap.md, which already records the same Phase 3.1-3.3 completions.
Proposed content
- Commit
scripts/run-recompute.shand add a short runbook subsection (deployment guide) covering the requiredADMIN_SECRET, thebatchargument, resume behavior, and theall=truebatch cap. - Decide the fate of
docs/UPDATE_SUMMARY.md: keep it as a dated record or delete it as duplicated bydocs/project-roadmap.md. - Commit the two review reports under
plans/reports/, since the recompute hardening issues cite them as their evidence source.
Acceptance criteria
-
scripts/run-recompute.shis tracked in git and referenced from a docs page -
docs/UPDATE_SUMMARY.mdis either committed or deleted - The two
plans/reports/review reports are committed or deleted - Documentation matches live behavior (verified against
apps/web/app/routes/api.admin.recompute.ts)
Notes
- Two further untracked entries,
.claude/session-state/andapps/web/.claude/session-state/, are session state and should stay ignored — they are out of scope here. - The review reports are the evidence base for the recompute hardening issues, so deleting them would remove the cited source unless it is summarised into the issues first.
- Related: PR #9 merged the recompute endpoint itself; this issue only closes the gap between the merged code and its operational instructions.
- Ngôn ngữ chính
- TypeScript
- Star
- 186
- Fork
- 40
- Merge trung bình
- 10 giờ 12 phút
- Pull request đã merge (30 ngày)
- 16
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của nextlevelbuilder/skillx
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
nextlevelbuilder/skillx#65 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
nextlevelbuilder/skillx#63 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
nextlevelbuilder/skillx#49 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
nextlevelbuilder/skillx#28 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
refactor
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
nextlevelbuilder/skillx#27 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của nextlevelbuilder/skillx
Issue tương tự
-
Create/import wallet form shows raw "password must be at least 1 characters" instead of the translated messageCó thể đã có người làm @CapThunder19 đã nhận hôm nay. Đang mởbug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
joinmarket-webui/jam#1538 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
t0ggles
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
nolimits4web/swiper#8240 · 1 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
modelcontextprotocol/inspector#2723 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
status:approved type:bug
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 70/100
egdev6/quorb#109 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày