Skip to content

refactor: only one pr context in storage.#32

Merged
knightedcodemonkey merged 3 commits intomainfrom
bananas
Mar 24, 2026
Merged

refactor: only one pr context in storage.#32
knightedcodemonkey merged 3 commits intomainfrom
bananas

Conversation

@knightedcodemonkey
Copy link
Owner

No description provided.

Copilot AI review requested due to automatic review settings March 24, 2026 14:12
Copy link
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 updates how the GitHub “Open PR” drawer persists draft PR configuration, aiming to keep only a single active PR context in localStorage, and adds an end-to-end test for that behavior.

Changes:

  • Add pruning logic so only one knighted:develop:github-pr-config:* entry remains in localStorage.
  • Add a Playwright test that verifies only a single PR context key is kept after switching repositories.
  • Includes small UI/style adjustments (toast positioning, token info icon sizing) and a minor refactor in type diagnostics loading.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/modules/github-pr-drawer.js Adds localStorage pruning to enforce a single active PR config context.
playwright/app.spec.ts Adds Playwright coverage to assert only one PR config key remains.
src/modules/type-diagnostics.js Refactors React type-file loading promise initialization.
src/styles/dialogs-overlays.css Moves toast positioning to bottom and adjusts entry transform direction.
src/styles/ai-controls.css Reduces GitHub token info button dimensions.

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

@knightedcodemonkey knightedcodemonkey merged commit 9cd4399 into main Mar 24, 2026
1 check passed
@knightedcodemonkey knightedcodemonkey deleted the bananas branch March 24, 2026 14:51
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