Skip to content

Fix workspace status aggregation logic#15

Merged
milldr merged 1 commit intomainfrom
feat/status-aggregation
Feb 27, 2026
Merged

Fix workspace status aggregation logic#15
milldr merged 1 commit intomainfrom
feat/status-aggregation

Conversation

@milldr
Copy link
Owner

@milldr milldr commented Feb 27, 2026

What

Two fixes to workspace status aggregation:

  1. Add a spec-level skip shell command that excludes repos from workspace aggregation (default check detects default-branch repos like main/master)
  2. Make the default: true status passive in aggregation — it no longer competes with active statuses like in-progress or in-review

Why

Default-branch repos always resolved to open (no PRs exist for main), dragging the entire workspace status down. And the fallback open status dominated active work signals, making workspace status misleading for multi-repo workspaces.

Ref

  • internal/status/types.go — new Skip field on SpecBody
  • internal/status/resolve.go — updated aggregation logic
  • internal/status/spec.go — default skip check for default-branch detection
  • internal/status/resolve_test.go — 5 new test cases

@milldr milldr force-pushed the feat/status-aggregation branch from 9731522 to bb88520 Compare February 27, 2026 19:17
@milldr milldr merged commit 0b5f8dc into main Feb 27, 2026
3 checks passed
@milldr milldr deleted the feat/status-aggregation branch February 27, 2026 19:18
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