-
Notifications
You must be signed in to change notification settings - Fork 2
docs(plan): Phase 79 residual authority — MASTER sole live index #1340
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,24 @@ | ||
| # Dependency Graph — Post-Polish Residual | ||
|
|
||
| ```mermaid | ||
| flowchart TD | ||
| subgraph P79[Phase 79 Docs Authority] | ||
| T791[T79.1 plan banners + MASTER] | ||
| T792[T79.2 perf gate evidence note] | ||
| T791 --> T792 | ||
| end | ||
|
|
||
| subgraph P80[Phase 80 Mobile hubClient] | ||
| T801[T80.1 inventory] | ||
| T802[T80.2 thin re-export] | ||
| T803[T80.3 RN boundary note] | ||
| T801 --> T802 --> T803 | ||
| end | ||
|
|
||
| T791 --> T801 | ||
| ``` | ||
|
|
||
| ## Parallelism | ||
|
|
||
| - B1 (docs) and B2 inventory (T80.1) can start after T79.1 lands; default serial for single-owner velocity. | ||
| - No file overlap between B1 and B2. |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,20 @@ | ||
| # Milestones — Post-Polish Residual Hardening | ||
|
|
||
| | # | Milestone | Criteria | Status | | ||
| |---|---|---|---| | ||
| | 79 | Docs Authority + Gates Hygiene | plan/* non-live; MASTER sole index; doc SSOT green; perf gate noted | open | | ||
| | 80 | Mobile hubClient Strangler | thin re-export; typecheck green; RN Hub-only boundary documented | open | | ||
|
|
||
| ## Adaptive control (initial) | ||
|
|
||
| ```yaml | ||
| adaptive: | ||
| drift_score: 0 | ||
| strategy: "strangler-mobile-hubclient-docs" | ||
| thresholds: | ||
| annotate: 1 | ||
| replan: 2 | ||
| rescope: 3 | ||
| total_tasks: 5 | ||
| completed_tasks: 0 | ||
| ``` | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,45 @@ | ||
| # Task Breakdown — Post-Polish Residual Hardening | ||
|
|
||
| > last-updated: 2026-07-20 | ||
| > program: post-visual-polish residual (after gate 89) | ||
| > tracking: GITHUB_FULL | ||
| > hard rule: **NO big-bang rewrite**; Mobile UI deep refactor out of scope | ||
| > strategy: Strangler Fig — thin mobile hubClient + docs authority | ||
|
|
||
| ## Overview | ||
|
|
||
| | Item | Value | | ||
| |---|---| | ||
| | Total Phases | 2 | | ||
| | Total Tasks | 5 | | ||
| | Strategy | Strangler + Spec-Driven | | ||
| | Super focus | R (Replaceable), P (Ports), E (Environment-agnostic) | | ||
|
|
||
| ## Phase 79: Docs Authority + Gates Hygiene | ||
|
|
||
| | # | Task | Priority | Effort | Depends | Super | Acceptance | | ||
| |---|---|---|---|---|---|---| | ||
| | T79.1 | Strengthen HISTORICAL banners on cleanup-baseline `docs/plan/*`; MASTER becomes sole live index for post-polish | P0 | S | — | E | plan files explicit non-live; MASTER links post-polish analysis; `verify-doc-ssot.ps1` green | | ||
| | T79.2 | Record backend perf gate green + optional workflow_dispatch note in MASTER/roadmap | P1 | S | T79.1 | E | evidence note; no claim of production capacity | | ||
|
|
||
| ## Phase 80: Mobile hubClient Strangler | ||
|
|
||
| | # | Task | Priority | Effort | Depends | Super | Acceptance | | ||
| |---|---|---|---|---|---|---| | ||
| | T80.1 | Inventory mobile-only surface (errors, WS helpers, fixture snapshot) vs shared SSOT methods | P0 | M | T79.1 | P | analysis note in issue/PR; list keep vs re-export | | ||
| | T80.2 | Thin mobile hubClient: re-export shared types/methods; keep mobile-only glue (SecureStore refresh, fixture snapshot, legacy WS types) | P0 | L | T80.1 | R,P | file shrinks; no new REST methods on mobile; `pnpm --filter agenthub-mobile-rn typecheck` (+ focused tests if present) | | ||
| | T80.3 | RN-safe boundary note: Mobile remains Hub-only; no Local Edge; document in AGENTS or mobile README | P1 | S | T80.2 | E,U | docs + existing verify scripts still pass | | ||
|
|
||
| ## Delivery Batches | ||
|
|
||
| | Batch | Tasks | Branch | Rationale | | ||
| |---|---|---|---| | ||
| | B1 | T79.1 + T79.2 | `feat/p79-docs-authority` | docs-only, independent | | ||
| | B2 | T80.1 + T80.2 + T80.3 | `feat/p80-mobile-hubclient-thin` | single mobile API surface PR for reviewability | | ||
|
|
||
| ## Out of Scope | ||
|
|
||
| - Live OIDC / secret rotation / packaged Desktop evidence | ||
| - Full Mobile UI redesign | ||
| - Further Edge handlers split without API change | ||
| - Static Visual QA gate chase past 89 |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,12 +1,13 @@ | ||
| # Task Breakdown — AgentHub Cleanup Baseline | ||
|
|
||
| > **HISTORICAL — cleanup-baseline freeze (closed 2026-07-16 / PR #446).** | ||
| > Not live backlog. Live phases and open issues: [../progress/MASTER.md](../progress/MASTER.md) (Phase 61 / milestone 82). | ||
| > **Not live backlog.** Do not resume Phases 1–7 or treat #424–#451 as open work. | ||
| > Live program SSOT: [../progress/MASTER.md](../progress/MASTER.md) (post-polish residual Phases 79–80 · issues #1335–#1339). | ||
| > Snapshot also under [../archives/cleanup-baseline/plan/](../archives/cleanup-baseline/plan/). | ||
| > | ||
| > last-updated: 2026-07-18 | ||
| > program: knowledge-first strangler cleanup + lightweight wiki (non-SSOT) | ||
| > tracking: GITHUB_FULL (Issues + Milestones + Project board) | ||
| > last-updated: 2026-07-21 | ||
| > program: knowledge-first strangler cleanup + lightweight wiki (non-SSOT) — **CLOSED** | ||
| > tracking: historical only; live tracking is GITHUB_FULL via MASTER + post-polish plan | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win Preserve MASTER as the sole live index. “Live tracking is GITHUB_FULL via MASTER + post-polish plan” makes the plan appear to be a second live tracker. Clarify that MASTER is the sole live index and the post-polish documents are linked plan SSOT artifacts. 🤖 Prompt for AI Agents |
||
| > hard rule: **NO big-bang rewrite** | ||
| > reality: Phases 1–7 complete (#424–#451 / PR #446). Do not list #447–#451 as open. | ||
|
|
||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,20 +1,29 @@ | ||
| # AgentHub Progress Tracker | ||
|
|
||
| > **Task**: continuous product polish (architecture / UIUX / design system / hygiene) | ||
| > **Started**: 2026-07-16 | ||
| > **Last Updated**: 2026-07-20 | ||
| > **Task**: post-polish residual hardening (docs authority + mobile hubClient strangler) | ||
| > **Started**: 2026-07-16 (Visual polish); residual program 2026-07-20 | ||
| > **Last Updated**: 2026-07-21 | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win Align all document timestamps with the actual publication date.
📍 Affects 4 files
🤖 Prompt for AI Agents |
||
| > **Mode**: `GITHUB_FULL` | ||
| > **Repo**: `TokenDanceLab/AgentHub` | ||
|
|
||
| ## Merged PRs (today, 19 total) | ||
| P74 #1295 · P75 #1310 · P76 #1315 · rescore #1317 · | ||
| P77 #1319 · P77 #1321 · P77 #1324 · P77 #1325 · | ||
| P78 #1326 · P78 #1327 · P78 #1328 · P78 #1329 · | ||
| P78 #1330 · P78 #1331 · infra #1332 · infra #1333 · | ||
| docs: rescore consolidation + AGENTS.md update + Phase 73/74 close | ||
| deps: dependabot serde_with bump #423 | ||
| ## Live program (sole index) | ||
|
|
||
| ## Milestones — ALL CLOSED | ||
| | Item | Value | | ||
| |---|---| | ||
| | Program | Post-Polish Residual Hardening | | ||
| | Analysis SSOT | [post-polish-project-overview](../analysis/post-polish-project-overview.md) · [module-inventory](../analysis/post-polish-module-inventory.md) · [risk-assessment](../analysis/post-polish-risk-assessment.md) | | ||
| | Plan SSOT | [task-breakdown](../plan/post-polish-task-breakdown.md) · [dependency-graph](../plan/post-polish-dependency-graph.md) · [milestones](../plan/post-polish-milestones.md) | | ||
| | Strategy | Strangler Fig — thin mobile hubClient + docs authority; **no** big-bang rewrite; **no** static Visual QA chase past 89 | | ||
| | Tracking | Issues **#1335–#1339** · GH milestones **98** (Phase 79) · **99** (Phase 80) | | ||
|
|
||
| ### Open phases | ||
|
|
||
| | Phase | Name | Milestone | Issues | Status | | ||
| |:------|:-----|:----------|:-------|:-------| | ||
| | 79 | Docs Authority + Gates Hygiene | 98 | #1335 #1336 | in progress | | ||
| | 80 | Mobile hubClient Strangler | 99 | #1337 #1338 #1339 | queued | | ||
|
|
||
| ### Closed polish phases (2026-07-20) | ||
|
|
||
| | Phase | Name | Status | | ||
| |:------|:-----|:-------| | ||
|
|
@@ -25,18 +34,16 @@ deps: dependabot serde_with bump #423 | |
| | 77 | Agents density + blank browser + terminal dock | closed | | ||
| | 78 | A11y focus + Glass border/shadows/elevation + CI path-filter | closed | | ||
|
|
||
| ## References | ||
| - [visual-qa-scorecard](../analysis/visual-qa-scorecard.md) — canonical SSOT (final gate 89) | ||
| - [visual-qa-score-2026-07-20-rescore-17-final.md](../analysis/visual-qa-score-2026-07-20-rescore-17-final.md) — full PR trace | ||
| - [_archive/](../analysis/_archive/) — 16 intermediate rescore files | ||
| Historical cleanup-baseline plan under `docs/plan/task-breakdown.md` (and siblings) is **HISTORICAL only** — not live backlog. | ||
|
|
||
| ## Final Status | ||
| ## Product tip & Visual QA | ||
|
|
||
| **Product tip**: `04a28663` | ||
| **Product tip**: see latest `master` (analysis tip at residual start: `4c222e06`) | ||
| **Gate**: **89**/100 — 🟢🟢🟢 **SHIP** | ||
| **Gate history**: 55 → 76 → 79 → 82 → 84 → 85 → 87 → 88 → **89** (+34 in one day) | ||
| **Gate history**: 55 → 76 → 79 → 82 → 84 → 85 → 87 → 88 → **89** | ||
|
|
||
| ### Dimension grid (7/9 maxed) | ||
|
|
||
| | Dim | Score | Max | Status | | ||
| |-----|-------|-----|--------| | ||
| | Glass | 18 | 18 | ✅ | | ||
|
|
@@ -49,16 +56,41 @@ deps: dependabot serde_with bump #423 | |
| | Type | 9 | 10 | ⏳ zh refinement | | ||
| | Motion | 9 | 10 | ⏳ interactive eval | | ||
|
|
||
| ### Infrastructure wins | ||
| - Unified CI path-filter (`changes` job via dorny/paths-filter@v3) | ||
| - Go-only PR skips frontend CI; CSS-only PR skips Go CI | ||
| - Estimated savings: up to 20 CI minutes per PR | ||
| References: [visual-qa-scorecard](../analysis/visual-qa-scorecard.md) · [rescore-17-final](../analysis/visual-qa-score-2026-07-20-rescore-17-final.md) · [_archive/](../analysis/_archive/) | ||
|
|
||
| ### Methodology ceiling | ||
| Remaining 3pt (Type/Motion/Empty) require interactive testing, multi-state data, | ||
| or multi-component CJK font changes — beyond static 1440×810 screenshot evaluation. | ||
|
|
||
| Remaining 3pt (Type/Motion/Empty) require interactive testing, multi-state data, or multi-component CJK font changes — beyond static 1440×810 screenshot evaluation. **Do not chase gate past 89** under residual program. | ||
|
|
||
| ## Infrastructure & gates hygiene | ||
|
|
||
| ### CI path-filter | ||
|
|
||
| - Unified `changes` job (`dorny/paths-filter@v3`) in `.github/workflows/checks.yml` | ||
| - Go-only PR skips frontend CI; CSS-only PR skips Go CI | ||
| - Estimated savings: up to ~20 CI minutes per PR | ||
|
|
||
| ### Backend perf / leak gates (T79.2 evidence) | ||
|
|
||
| | Item | State | Note | | ||
| |---|---|---| | ||
| | `scripts/verify/verify-backend-perf-leak-gates.ps1` | **PASS** (behavior + short microbench) | Not production capacity | | ||
| | [backend-performance-gates.md](../reference/backend-performance-gates.md) | Active owner (dated 2026-06-27) | Evidence classes: behavior / microbench / load smoke / pprof | | ||
| | Capacity claim | **Not claimed** | Load smoke / pprof still path-specific; no “production capacity proven” language | | ||
|
Comment on lines
+73
to
+79
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win Make the perf/leak evidence boundary explicit in every live summary.
📍 Affects 2 files
🤖 Prompt for AI Agents |
||
|
|
||
| Optional future: wire script as `workflow_dispatch` only — not every PR (see residual risk assessment T5). | ||
|
|
||
| ## Explicit out of scope (residual) | ||
|
|
||
| - Live OIDC / secret rotation / packaged Desktop evidence | ||
| - Full Mobile UI redesign | ||
| - Static Visual QA gate chase past 89 | ||
| - Edge handlers further split without concrete API change | ||
|
|
||
| ## Session Log | ||
|
|
||
| | Date | Summary | | ||
| |:-----|:--------| | ||
| | 2026-07-20 | 🟢🟢🟢 **FINAL Ship 89** — 19 commits, 7/9 dims maxed, CI optimized, docs consolidated | | ||
| | 2026-07-20 | 🟢🟢🟢 **Ship 89** — polish Phases 73–78 closed; CI path-filter; docs rescore archive | | ||
| | 2026-07-20 | SDD Phase 0–1: post-polish analysis trio committed | | ||
| | 2026-07-21 | Residual program live: MASTER sole index; Phases 79–80 · #1335–#1339 | | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win
Keep phase numbers separate from GitHub milestone numbers.
This artifact labels
79and80as milestones, whiledocs/progress/MASTER.mdanddocs/plan/milestones.mddefine GitHub milestones 98 and 99 for Phases 79–80. Add explicitPhaseandGH milestonecolumns, or change the identifiers to 98/99.🤖 Prompt for AI Agents