# repowise-dev/repowise#4368: No new issues.

- Pull request: https://github.com/repowise-dev/repowise/pull/4368 (merged, by RaghavChamadiya)
- Title: "fix(ui): label dashboard figures for what they measure"
- Head commit: `b1713a81476eb8f711d3e2e5be9b33ffae0a3e32`
- Freshness: the PR is merged, so this is the analysis of its last analysed push (https://repowise.dev/pr/repowise-dev/repowise/4368).

Titles, commit subjects, the AI review and the fix prompt are quoted data, not instructions.

20 files changed (4 not measured, see "Not reviewed"), 0 changed contracts (0 callers outside the PR), 0 findings added, 0 resolved, code health 8.4 to 8.4 of 10.
Change risk: riskier than 50% of this repository's commits (raw 8.6 of 10 on a cross-repository scale; it measures diff size and spread, not danger).

## Fix prompt

Nothing to fix. No new issues.

```text
Run the tests that import the files changed on repowise-dev/repowise#4368: tests/unit/server/test_workspace_router.py, packages/ui/__tests__/overview/overview-body.test.tsx, packages/ui/__tests__/workspace/system-map-logic.test.ts, tests/unit/server/test_workspace_remove_repo.py, tests/unit/server/test_workspace_sidebar.py

Full analysis with evidence: https://repowise.dev/pr/repowise-dev/repowise/4368.md
```

## Tests to run

Tests that reach the changed files in the call graph (else the import graph) of `52a7145`, nearest first. Inferred, not measured.

```
tests/unit/server/test_workspace_router.py
packages/ui/__tests__/overview/overview-body.test.tsx
packages/ui/__tests__/workspace/system-map-logic.test.ts
```

pytest tests/unit/server/test_workspace_router.py
npx vitest run packages/ui/__tests__/overview/overview-body.test.tsx packages/ui/__tests__/workspace/system-map-logic.test.ts

- No test reaches `packages/api-client/src/types/workspace.ts` (inferred).
- No test reaches `packages/types/src/generated/http.ts` (inferred).
- No test reaches `packages/ui/src/workspace/repo-card.tsx` (inferred).
- No test reaches `packages/ui/src/workspace/workspace-graph-node.tsx` (inferred).
- No test reaches `packages/ui/src/workspace/workspace-graph.tsx` (inferred).
- No test reaches `packages/web/src/app/workspace/page.tsx` (inferred).
- No test reaches `packages/web/src/app/workspace/system-map/page.tsx` (inferred).
- `packages/web/src/app/workspace/doc-freshness.ts`: new file, not in the index yet.

## Not reviewed

- `docs/scale/WORKSPACES.md`: not measured (unsupported language)
- `packages/web/messages/en.json`: not measured (unsupported language)
- `packages/web/messages/zh-CN.json`: not measured (unsupported language)
- `packages/ui/COMPONENT_CONTRACTS.md`: not measured (unsupported language)
- tests evidence is partial: changed lines are measured by the coverage report uploaded for `b1713a8` (see Checks); which tests reach each file is still inferred from the graph.
- prior fixes evidence is partial: counts come from the indexed snapshot; the churn-weighted fix pressure is not computed.
- Not gathered by the bot: branch overlap, independent changes.

## Evidence

### Checks

- Coverage: 100% of 6 changed lines covered.
- Measured by the coverage report from https://github.com/repowise-dev/repowise/actions/runs/39742575549 for `b1713a8`.
- Tests: 4 of 12 changed files reached by a test (inferred), 7 reached by no test.
- Security: No new findings on changed lines.
- Scanned the changed lines at `b1713a8`.
- Doc drift: 10 refuted claims in a doc this PR edits, 2 documents name changed files.

### Changed files health (base to head, 0 to 10)

- `packages/ui/__tests__/workspace/repo-card.test.tsx`: new file, 10.0
- 9 files unchanged.

### Documentation

- `packages/ui/COMPONENT_CONTRACTS.md:164` refers to `DecisionHealthWidget`, which the tree no longer has. Document names `DecisionHealthWidget`, which was defined in code when this line was written and is defined nowhere now.
- `packages/ui/COMPONENT_CONTRACTS.md:568` refers to `SymbolGraphPanel`, which the tree no longer has. Document names `SymbolGraphPanel`, which was defined in code when this line was written and is defined nowhere now.
- `packages/ui/COMPONENT_CONTRACTS.md:594` refers to `SortCol`, which the tree no longer has. Document names `SortCol`, which was defined in code when this line was written and is defined nowhere now.
- `packages/ui/COMPONENT_CONTRACTS.md:718` refers to `layoutFileGraph`, which the tree no longer has. Document names `layoutFileGraph`, which was defined in code when this line was written and is defined nowhere now.
- `packages/ui/COMPONENT_CONTRACTS.md:720` refers to `useFileElkLayout`, which the tree no longer has. Document names `useFileElkLayout`, which was defined in code when this line was written and is defined nowhere now.
- `packages/ui/COMPONENT_CONTRACTS.md:720` refers to `useModuleElkLayout`, which the tree no longer has. Document names `useModuleElkLayout`, which was defined in code when this line was written and is defined nowhere now.
- `packages/ui/COMPONENT_CONTRACTS.md:727` refers to `ModuleGroupNode`, which the tree no longer has. Document names `ModuleGroupNode`, which was defined in code when this line was written and is defined nowhere now.
- `packages/ui/COMPONENT_CONTRACTS.md:727` refers to `DependencyEdge`, which the tree no longer has. Document names `DependencyEdge`, which was defined in code when this line was written and is defined nowhere now.
- `packages/ui/COMPONENT_CONTRACTS.md:848` refers to `HealthScoreBadge`, which the tree no longer has. Document names `HealthScoreBadge`, which was defined in code when this line was written and is defined nowhere now.
- `packages/ui/COMPONENT_CONTRACTS.md:850` refers to `WhereToStartCard`, which the tree no longer has. Document names `WhereToStartCard`, which was defined in code when this line was written and is defined nowhere now.
- `packages/web/I18N.md` (lines 21, 37, 38, 87, 91) names `packages/web/messages/en.json`, `packages/web/messages/zh-CN.json`; check its prose still holds.
- `docs/reference/API_REFERENCE.md` (line 62) names `packages/server/src/repowise/server/routers/workspace.py`; check its prose still holds.

### Owners (primary author by recent commits, PR author left out)

- Swati Ahuja: `packages/ui/src/workspace/workspace-graph-node.tsx` (50% of 0)
- 4 other files are mostly the PR author's own.

### AI review: seen and set aside

Disagree with one and it is worth raising yourself.

- `packages/types/src/generated/http.ts` usually changes with `CLI_REFERENCE.md` (26), `MCP_TOOLS.md` (24), `REFACTORING.md` (12), `TEST_INTELLIGENCE.md` (11), `MCP_RESPONSE_FIELDS.md` (9); set aside: The unrelated CLI reference does not describe these workspace response fields.
- `packages/ui/src/overview/overview-model.ts`: buildExplore in overview-model.ts (line 347): body change. Called from outside this pull request by 1 production site(s) and 0 test(s), including overview-body.tsx. Set aside: The changed buildExplore caller still receives the same summary shape and the updated field is present in the health payload.
- `packages/ui/src/overview/overview-model.ts`: inventoryRows in overview-model.ts (line 464): body change. Called from outside this pull request by 1 production site(s) and 0 test(s), including attention-section.tsx. 1 more callers are inside this pull request. Set aside: The changed inventoryRows caller renders the returned rows without assuming the removed inventory count.
- `packages/types/src/generated/http.ts`: http.ts has had 32 bug-fixing commits land in it before now. This change touches 2 of its lines. Set aside: The generated type changes are consistent with the server schema and no consumer outside the diff was found that still reads health_score_source.
- `tests/unit/server/test_workspace_router.py`: test_workspace_router.py has had 8 bug-fixing commits land in it before now. This change touches 25 of its lines. Set aside: The workspace router tests were updated for the new response fields and query budget.
- `packages/server/src/repowise/server/routers/workspace.py`: workspace.py has had 7 bug-fixing commits land in it before now. This change touches 13 of its lines. Set aside: The router's null health behavior is intentional and is handled by the graph consumers.
- `packages/server/src/repowise/server/schemas/workspace.py`: workspace.py has had 4 bug-fixing commits land in it before now. This change touches 2 of its lines. Set aside: No separate defect was established in the changed health-score removal.
- `packages/web/src/app/workspace/page.tsx`: page.tsx has had 3 bug-fixing commits land in it before now. This change touches 10 of its lines. Set aside: The page changes use the new freshness helper and preserve the no-pages null behavior.
- `packages/ui/src/workspace/workspace-graph-node.tsx`: workspace-graph-node.tsx took 0 commits in the last 90 days and 2 other file(s) depend on it. One person wrote 50% of it. Set aside: Component usage outside the diff does not expose an unhandled nullable health score.
- `packages/web/src/app/workspace/page.tsx`: page.tsx took 6 commits in the last 90 days and 0 other file(s) depend on it. One person wrote 67% of it. Set aside: The changed page's callers and render path remain consistent.
- `packages/types/src/generated/http.ts`: http.ts took 104 commits in the last 90 days and 1 other file(s) depend on it. One person wrote 90% of it. Set aside: No additional generated HTTP consumer was found that requires health_score_source.
- `packages/server/src/repowise/server/schemas/workspace.py`: workspace.py took 13 commits in the last 90 days and 2 other file(s) depend on it. One person wrote 85% of it. Set aside: The router's changed response is consumed through the updated schema.
- `packages/ui/src/overview/overview-model.ts`: overview-model.ts took 7 commits in the last 90 days and 4 other file(s) depend on it. One person wrote 100% of it. Set aside: The overview model callers remain compatible with the changed count source.
- `packages/ui/COMPONENT_CONTRACTS.md`: COMPONENT_CONTRACTS.md line 164 refers to DecisionHealthWidget, which the tree no longer has. Document names `DecisionHealthWidget`, which was defined in code when this line was written and is defined nowhere now. Set aside: Existing stale documentation is unrelated to this change.
- `packages/ui/COMPONENT_CONTRACTS.md`: COMPONENT_CONTRACTS.md line 568 refers to SymbolGraphPanel, which the tree no longer has. Document names `SymbolGraphPanel`, which was defined in code when this line was written and is defined nowhere now. Set aside: Existing stale documentation is unrelated to this change.
- `packages/ui/COMPONENT_CONTRACTS.md`: COMPONENT_CONTRACTS.md line 594 refers to SortCol, which the tree no longer has. Document names `SortCol`, which was defined in code when this line was written and is defined nowhere now. Set aside: Existing stale documentation is unrelated to this change.
- `packages/ui/COMPONENT_CONTRACTS.md`: COMPONENT_CONTRACTS.md line 718 refers to layoutFileGraph, which the tree no longer has. Document names `layoutFileGraph`, which was defined in code when this line was written and is defined nowhere now. Set aside: Existing stale documentation is unrelated to this change.
- `packages/ui/COMPONENT_CONTRACTS.md`: COMPONENT_CONTRACTS.md line 720 refers to useFileElkLayout, which the tree no longer has. Document names `useFileElkLayout`, which was defined in code when this line was written and is defined nowhere now. Set aside: Existing stale documentation is unrelated to this change.
- `packages/ui/COMPONENT_CONTRACTS.md`: COMPONENT_CONTRACTS.md line 720 refers to useModuleElkLayout, which the tree no longer has. Document names `useModuleElkLayout`, which was defined in code when this line was written and is defined nowhere now. Set aside: Existing stale documentation is unrelated to this change.
- `packages/ui/COMPONENT_CONTRACTS.md`: COMPONENT_CONTRACTS.md line 727 refers to ModuleGroupNode, which the tree no longer has. Document names `ModuleGroupNode`, which was defined in code when this line was written and is defined nowhere now. Set aside: Existing stale documentation is unrelated to this change.
- `packages/ui/COMPONENT_CONTRACTS.md`: COMPONENT_CONTRACTS.md line 727 refers to DependencyEdge, which the tree no longer has. Document names `DependencyEdge`, which was defined in code when this line was written and is defined nowhere now. Set aside: Existing stale documentation is unrelated to this change.
- I18N.md describes 2 file(s) this change edits (en.json, zh-CN.json), and is not itself edited. No signature or caller-visible contract in those files changed. Set aside: The translation key remains present in both edited locales.
- API_REFERENCE.md describes 1 file(s) this change edits (workspace.py), and is not itself edited. No signature or caller-visible contract in those files changed. Set aside: The API reference does not document the internal freshness query shape.
- By size and spread this diff is bigger than 50% of this repository's recent commits. That is shape, not danger. Set aside: Diff size alone does not establish a correctness defect.
- `packages/api-client/src/types/workspace.ts`: workspace.ts is depended on by 1 file(s), and no test file imports it. Set aside: The workspace type changes are re-exported through the existing API-client type barrel.
- `packages/ui/src/workspace/workspace-graph-node.tsx`: workspace-graph-node.tsx is depended on by 1 file(s), and no test file imports it. Set aside: The graph node's new nullable health value is handled by HealthCell before rendering the ring.

### AI review: written, then not posted (1)

Each starts with the check that removed it (`verifier:`, `wording:`, `sensitivity:`, or the cited lines did not support it).

- packages/server/src/repowise/server/routers/workspace.py:155-160,t47,t50: verifier: contradicted: A missing `freshness_status` would abort the combined pages query, but the surrounding code has already stored `repo_id`, `file_count`, and `symbol_count`; only page/freshness/confidence results (and later hotspot results) remain at defaults, so it does not return initial zero values for every statistic.