Skip to content

feat: show workspace context status by default, allow rename.#130

Merged
knightedcodemonkey merged 2 commits intomainfrom
bananas
May 3, 2026
Merged

feat: show workspace context status by default, allow rename.#130
knightedcodemonkey merged 2 commits intomainfrom
bananas

Conversation

@knightedcodemonkey
Copy link
Copy Markdown
Owner

  • This uses window.prompt as a first pass.

Copilot AI review requested due to automatic review settings May 3, 2026 21:11
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR makes the workspace context status row always present in the main grid (instead of being gated behind PAT connection) and adds a rename action for local stored workspaces via the Workspaces drawer.

Changes:

  • Always render the workspace context status row in the app layout and show it by default.
  • Add a “Rename” button to the Workspaces drawer, wiring it through the GitHub workflows to update stored local workspace metadata.
  • Simplify workspace option labels (remove the local: prefix) and update Playwright expectations accordingly.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/styles/layout-shell.css Makes the workspace context row a permanent part of the app grid layout.
src/index.html Shows the workspace context status element by default; adds the Workspaces “Rename” button.
src/modules/app-core/workspace-context-status-controller.js Updates status rendering logic to always show the status node and adjust repository display behavior.
src/modules/workspace/workspaces-drawer/drawer.js Adds rename button handling + updates option label formatting.
src/modules/app-core/github-workflows.js Implements the local workspace rename flow (currently via window.prompt).
src/app.js Wires rename button + passes persisted workspace metadata to status controller.
playwright/github-pr-drawer/open-pr-create.spec.ts Updates label assertions after removing local: prefix.
playwright/github-byot-ai.spec.ts Updates status visibility test and adds a rename flow test using prompt dialog handling.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/modules/app-core/workspace-context-status-controller.js Outdated
Comment thread src/modules/app-core/github-workflows.js
@knightedcodemonkey knightedcodemonkey merged commit 47cc8eb into main May 3, 2026
6 checks passed
@knightedcodemonkey knightedcodemonkey deleted the bananas branch May 3, 2026 21:33
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.

2 participants