Skip to content

Make Active Agents tree distinguish OpenSpec workflow nodes#380

Merged
NagyVikt merged 1 commit intomainfrom
agent/codex/add-openspec-and-provider-icons-2026-04-23-16-49
Apr 23, 2026
Merged

Make Active Agents tree distinguish OpenSpec workflow nodes#380
NagyVikt merged 1 commit intomainfrom
agent/codex/add-openspec-and-provider-icons-2026-04-23-16-49

Conversation

@NagyVikt
Copy link
Copy Markdown
Collaborator

Automated by gx branch finish (PR flow).

The companion already shipped semantic OpenSpec icons in Explorer, but the Active Agents raw tree still flattened those nodes to generic file and folder icons. This reuses the bundled icon manifest for tree items, preserves warning overrides, mirrors the template source, and adds focused regression coverage plus a follow-up OpenSpec lane.

Constraint: Keep warning and lock icons higher priority than workflow art
Rejected: Rely on the Explorer file icon theme alone | Active Agents tree rows still need explicit iconPath resolution
Confidence: high
Scope-risk: narrow
Directive: Keep the file-icon manifest and tree-item resolver in sync when adding new workflow file names
Tested: node --test test/vscode-active-agents-session-state.test.js
Tested: openspec validate agent-codex-add-openspec-and-provider-icons-2026-04-23-16-49 --type change --strict
Tested: openspec validate --specs
Not-tested: Manual VS Code desktop install/reload check
@NagyVikt NagyVikt merged commit c78d520 into main Apr 23, 2026
@NagyVikt NagyVikt deleted the agent/codex/add-openspec-and-provider-icons-2026-04-23-16-49 branch April 23, 2026 14:53
NagyVikt added a commit that referenced this pull request Apr 23, 2026
The feature lane for Active Agents OpenSpec tree icons already merged via PR #380, but its tasks file still showed cleanup boxes as incomplete. This follow-up records the real PR metadata and prune evidence without reopening product code.

Constraint: Do not reopen shipped extension behavior for bookkeeping-only cleanup evidence
Rejected: Leave the merged tasks file unchecked | it makes finish state on main untrustworthy
Confidence: high
Scope-risk: narrow
Directive: When finish merges before tasks.md is updated, land a tiny follow-up that records exact PR and prune evidence
Tested: gh pr view agent/codex/add-openspec-and-provider-icons-2026-04-23-16-49 --json url,state,mergedAt,mergeCommit
Tested: git branch --list agent/codex/add-openspec-and-provider-icons-2026-04-23-16-49
Tested: git branch -r --list origin/agent/codex/add-openspec-and-provider-icons-2026-04-23-16-49
Tested: git worktree list --porcelain
Not-tested: Product behavior; docs-only backfill

Co-authored-by: NagyVikt <nagy.viktordp@gmail.com>
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