Skip to content

feat: release ai chat.#56

Merged
knightedcodemonkey merged 1 commit intomainfrom
bananas
Apr 1, 2026
Merged

feat: release ai chat.#56
knightedcodemonkey merged 1 commit intomainfrom
bananas

Conversation

@knightedcodemonkey
Copy link
Copy Markdown
Owner

@knightedcodemonkey knightedcodemonkey commented Apr 1, 2026

Copilot AI review requested due to automatic review settings April 1, 2026 02:43
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 removes the opt-in feature flag for AI chat and makes the GitHub AI chat drawer available by default once a GitHub token is connected, aligning docs and Playwright coverage with the new default behavior.

Changes:

  • Remove featureEnabled gating from the GitHub chat drawer and token-visibility logic so chat is token-driven.
  • Delete the feature-flags module and remove ?feature-ai=true documentation references.
  • Update Playwright specs to load the app without the feature-ai query parameter and adjust expected chat visibility.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/modules/github-chat-drawer/drawer.js Removes feature-flag early return so the drawer logic is always initialized.
src/modules/feature-flags.js Removes the AI assistant feature-flag implementation entirely.
src/app.js Removes feature-flag wiring; shows chat toggle when a token is present.
README.md Removes outdated note about opting into AI chat via ?feature-ai=true.
docs/byot.md Updates BYOT docs to reflect AI chat availability after token connection (no flag).
playwright/github-pr-drawer.spec.ts Drops ?feature-ai=true from app loads across PR drawer tests.
playwright/github-byot-ai.spec.ts Updates test expectations and drops feature-flag-specific test coverage.

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

@knightedcodemonkey knightedcodemonkey merged commit af26c6e into main Apr 1, 2026
9 checks passed
@knightedcodemonkey knightedcodemonkey deleted the bananas branch April 1, 2026 02:48
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.

Feature: Add an AI Agent Side Panel using GitHub Models for @knighted/develop

2 participants