Refactor and rethink of the plan attributes

EduarD3V commented

2026-10-03 00:53:51 +02:00

Owner

Right now, under http://100.118.73.36:5173/plans/UUID there are many many attributes. Check which ones can we deprecate and not use any more (delete) by assuming the user is not an idiot and has a shared org or a personal org

EduarD3V added this to the Gotoi project

2026-10-03 00:54:52 +02:00

EduarD3V referenced this issue from a pull request that will close it,

2026-10-03 01:22:39 +02:00

Issue #1: Refactor and rethink of the plan attributes #2

EduarD3V commented

2026-10-03 01:22:50 +02:00

Author

Owner

Working branch pushed and PR opened: #2 (targets develop).

Branch: issue/1-plan-attributes · commits: 1 (579562d dashboard: drop redundant plan detail attributes, 2 files, +6/−22)

Removed from /plans/[id]: full Id (already the route param), Commit (opaque hash, always empty for steps plans), Kind (raw enum that drives agent-role scoring, not an operator decision), and the audit-only ownerId badge (ADR-005: the org is the only owner key). UC-FE-41 updated to match.

Gates — all exit 0: docs/use-cases/_check.py (346 UCs conform) · scripts/check-ai-headers.py (529 ok) · scripts/check-type-mirror.py · dashboard bun run test (230 passed) · dashboard bun run check (0 errors, 7 pre-existing warnings) · dashboard bun run build. No Showing lines footer in source.

Presentation-only; middleware model/schema/API untouched. Live smoke against an isolated stack confirmed the facts card renders Status/Health/Compat/Version/Trust/Autonomy with no Id/Commit/Kind/owner badge.

Needs a human: (1) Kind is functional in coordinator.ts — restore that row if the page should advertise remediation vs diagnostic; (2) ownerId badge is still rendered on /plans, sources, installs and pinned by tests — out of scope here; (3) UC-FE-41 drift fixed (it listed a Share button that does not exist).

Not merging — a human merges into develop.

Working branch pushed and PR opened: https://git.eduard3v.com/EduarD3V/gotoi/pulls/2 (targets `develop`). Branch: `issue/1-plan-attributes` · commits: 1 (`579562d` dashboard: drop redundant plan detail attributes, 2 files, +6/−22) **Removed from /plans/[id]:** full `Id` (already the route param), `Commit` (opaque hash, always empty for steps plans), `Kind` (raw enum that drives agent-role scoring, not an operator decision), and the audit-only `ownerId` badge (ADR-005: the org is the only owner key). UC-FE-41 updated to match. **Gates — all exit 0:** `docs/use-cases/_check.py` (346 UCs conform) · `scripts/check-ai-headers.py` (529 ok) · `scripts/check-type-mirror.py` · `dashboard bun run test` (230 passed) · `dashboard bun run check` (0 errors, 7 pre-existing warnings) · `dashboard bun run build`. No `Showing lines` footer in source. Presentation-only; middleware model/schema/API untouched. Live smoke against an isolated stack confirmed the facts card renders Status/Health/Compat/Version/Trust/Autonomy with no Id/Commit/Kind/owner badge. **Needs a human:** (1) `Kind` is functional in `coordinator.ts` — restore that row if the page should advertise remediation vs diagnostic; (2) `ownerId` badge is still rendered on /plans, sources, installs and pinned by tests — out of scope here; (3) UC-FE-41 drift fixed (it listed a Share button that does not exist). Not merging — a human merges into `develop`.

EduarD3V added reference issue/1-plan-attributes

2026-10-04 21:45:03 +02:00

EduarD3V commented

2026-10-04 21:50:36 +02:00

Author

Owner

This can be closed, is ok

EduarD3V

2026-10-04 21:50:37 +02:00