Skip to content

[Bugfix #1169] Collapse Pull Requests / Recently Closed / Team / Status sidebar views by default#1171

Merged
amrmelsayed merged 8 commits into
mainfrom
builder/bugfix-1169
Jul 13, 2026
Merged

[Bugfix #1169] Collapse Pull Requests / Recently Closed / Team / Status sidebar views by default#1171
amrmelsayed merged 8 commits into
mainfrom
builder/bugfix-1169

Conversation

@amrmelsayed

@amrmelsayed amrmelsayed commented Jul 13, 2026

Copy link
Copy Markdown
Collaborator

Summary

Adds "visibility": "collapsed" to four lower-priority Codev VS Code sidebar views so they render as ~22px headers on first install instead of fully expanded. Reclaims vertical space for the primary workload surfaces (Workspace, Agents, Backlog).

Root Cause

In packages/vscode/package.json, all seven contributes.views.codev[] entries omitted "visibility", so VS Code applied its default (visible). VS Code exposes no manifest lever for view height ratios, so the only way to bias the first-render distribution toward the primary surfaces is defaulting lower-priority views to collapsed.

Fix

Added "visibility": "collapsed" to the four lower-priority views, preserving each existing when clause:

  • codev.pullRequests (Pull Requests)
  • codev.recentlyClosed (Recently Closed)
  • codev.team (Team)
  • codev.status (Status)

Workspace, Agents, and Backlog stay expanded. Uses collapsed (not hideByDefault) so discoverability is preserved: the header row stays visible, one click to expand. Follows the existing precedent set by codev.placeholder in the codevPanel container.

Scope discipline: manifest + test only, no source changes. Release-notes entries (CHANGELOG.md [Unreleased] + docs/releases/UNRELEASED.md Polish) are handled on the repo's dedicated docs/vscode-changelog branch per the established dual-accumulate workflow, not on this code branch.

Caveat for existing users

visibility: "collapsed" only applies to first-install / new-workspace state. Users who have already interacted with these views keep their persisted layout, and can manually collapse if they want the new distribution.

Test Plan

  • Regression test added to packages/vscode/src/__tests__/contributes-view-gating.test.ts (the existing vscode: IDE-mode foundation: dual-mode activation, workspace/IDE context keys, empty-window surfaces #1144 manifest-invariant suite): asserts the four lower-priority views contribute "visibility": "collapsed" and the three primary views keep no override. Fails without the fix, passes with it. pnpm vitest run → 21 passed.
  • npm run build — passes (porch checks: build ✓, tests ✓)
  • JSON validity of package.json confirmed
  • Manual: VS Code Extension Development Host smoke on a fresh workspace shows the four views collapsed to headers, primary views expanded.

CMAP Review

  • Gemini: APPROVE (HIGH)
  • Claude: APPROVE (HIGH)
  • Codex: REQUEST_CHANGES (HIGH) → resolved by adding the manifest-invariant regression test above.

Fixes #1169

… Status views by default

Add "visibility": "collapsed" to the four lower-priority Codev sidebar
views so they render as headers on first install, reclaiming vertical
space for the primary Workspace / Agents / Backlog surfaces. Existing
"when" clauses preserved. Applies to first-install/new-workspace state
only; persisted user state is unaffected.
…y views

Add manifest-invariant assertions (matching the existing #1144 view-gating
tests) that codev.pullRequests / recentlyClosed / team / status contribute
"visibility": "collapsed" and the primary views keep no override. Addresses
CMAP (codex) feedback: the repo already protects manifest contracts with
Vitest, so this bug is testable and should be locked down.
@amrmelsayed amrmelsayed merged commit 57de0fa into main Jul 13, 2026
6 checks passed
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.

vscode: collapse Pull Requests / Recently Closed / Team / Status by default to reclaim sidebar vertical space

1 participant