Skip to content

fix(starpuff): 收斂 Codex review 教學與治理缺口 - #1047

Merged
s123104 merged 8 commits into
mainfrom
fix/starpuff-review-followups
Aug 26, 2026
Merged

fix(starpuff): 收斂 Codex review 教學與治理缺口#1047
s123104 merged 8 commits into
mainfrom
fix/starpuff-review-followups

Conversation

@s123104

@s123104 s123104 commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

摘要

這是已合併 PR #1045 的 follow-up,收斂合併後新增的 Codex review 意見,並將歷史 PR #1039#1031 的治理缺口補回可驗證的程式與文件。

變更

  • Shelly 首次進入旋轉殼盾時發出既有 feature event,讓殼盾情境提示可實際進入下砸步驟。
  • 移除不符合教學目標的玩家殼盾事件監聽。
  • 操作提示開關即時套用到目前場景,關閉清除提示與焦點,重新開啟恢復當前步驟。
  • MoneyBox 停供 issue 補上 bug 標籤與摘要、背景/證據、影響、範圍、驗收標準。
  • AGENTS.md 同步 MoneyBox 升級節流 SSOT。
  • brace-expansion 測試改為精確驗證 >=3.0.0 覆蓋。
  • 新增 patch changeset,002 記錄集中於單一提交。

驗證

  • workspace pre-commit:typecheck、Prettier、002 守門通過。
  • workspace pre-push:全 workspace typecheck、unit tests、root tests、build:ratewise 通過。
  • StarPuff targeted tests:3 files、11 tests 通過。
  • RateWise build-scripts targeted tests:49 tests 通過。
  • StarPuff build 通過。
  • E2E:依需求略過;CI 若排程,非必需 E2E 不作為本次合併條件。

關聯

haotool added 7 commits August 26, 2026 13:43
- Shelly 首次進入 spin 時送出實際教學事件
- 補回歸測試鎖定事件與狀態轉換

測試:pnpm --filter @app/starpuff exec vitest run src/game/systems/enemies.test.ts
- 教學只接受 Shelly 進入 spin 的實際事件
- 移除可能誤完成 lesson 的玩家護盾事件監聽

測試:starpuff guidance/enemies targeted Vitest
- 訂閱設定變更並即時隱藏或恢復目前提示
- 補測試鎖定同一場景的開關行為

測試:starpuff guidanceDirector targeted Vitest
- 自動事故 issue 補上 bug 類型與正式追蹤段落
- 以 source-level test 鎖定 issue 欄位與驗收契約

測試:ratewise build-scripts targeted Vitest
- 直接驗證第三條 override 從 >=3.0.0 起算
- 防止只驗到 5.x 而漏掉 3.x/4.x 依賴

測試:pnpm --filter @app/ratewise exec vitest run src/config/__tests__/build-scripts.test.ts
- 將排程中斷的 issue 標籤與正式 body 契約寫入 AGENTS
- 保留查詢失敗時的 fail-safe 與恢復關閉語意

測試:node scripts/verify-002-log.mjs
- 新增 StarPuff follow-up patch changeset
- 集中記錄本 PR 五項修正與一項已驗證的 neutral 意見

測試:node scripts/verify-002-log.mjs
@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Deprecation Warning: The deny-licenses option is deprecated for possible removal in the next major release. For more information, see issue 997.

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@github-actions

Copy link
Copy Markdown
Contributor

✅ SEO 審計通過!所有 2026 標準驗證項目都符合要求。

  • ✅ Sitemap 2026 標準
  • ✅ Breadcrumb Schema
  • ✅ JSON-LD 結構化數據
  • ✅ 內部連結結構

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e2833cc768

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .changeset/shelly-guidance-review-fix.md
Comment thread AGENTS.md
- 讓 follow-up changeset 同時涵蓋 RateWise 與 StarPuff package
- 將 MoneyBox issue 標籤與五段 body 契約同步至 CLAUDE.md

測試:pnpm changeset status;node scripts/verify-002-log.mjs;StarPuff 3 files/11 tests;RateWise 49 tests
@github-actions

Copy link
Copy Markdown
Contributor

✅ SEO 審計通過!所有 2026 標準驗證項目都符合要求。

  • ✅ Sitemap 2026 標準
  • ✅ Breadcrumb Schema
  • ✅ JSON-LD 結構化數據
  • ✅ 內部連結結構

@s123104
s123104 merged commit c468613 into main Aug 26, 2026
17 of 19 checks passed
@s123104
s123104 deleted the fix/starpuff-review-followups branch August 26, 2026 06:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant