fix(ui): prevent browser autofill in AI Config modal + E2E verification for PR #22 #23
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add multi-provider AI service architecture supporting OpenAI, Groq, Anthropic, local Ollama, and custom endpoints - Implement unified AI service abstraction layer with automatic provider switching - Create user-friendly configuration interface with real-time validation - Add persistent configuration storage in localStorage - Maintain backward compatibility with existing Portkey integration - Enhance error handling and fallback mechanisms - Update documentation and add environment variable examples New files: - nextjs-web-app/src/lib/ai-config.ts - AI provider configuration management - nextjs-web-app/src/lib/ai-service.ts - Unified AI service abstraction - nextjs-web-app/src/hooks/useAIConfig.ts - Configuration management hook - nextjs-web-app/src/components/AIConfigModal.tsx - Configuration UI modal - nextjs-web-app/src/components/AIConfigButton.tsx - Configuration button component - nextjs-web-app/.env.example - Environment variables template - nextjs-web-app/src/lib/__tests__/ai-config.test.ts - Unit tests Modified files: - nextjs-web-app/src/app/api/generate/route.ts - Multi-provider API support - nextjs-web-app/src/app/page.tsx - AI configuration integration - nextjs-web-app/src/app/results/page.tsx - Provider selection support - README.md - Updated documentation with new features Features: - Support for 6 different AI providers out of the box - Seamless provider switching without application restart - Real-time configuration validation with user feedback - Secure API key storage and management - Full TypeScript support with proper type safety - Comprehensive error handling and user-friendly messages - No breaking changes - fully backward compatible
…dental credential prefill\n\n- Add autocomplete, name, and ignore attributes to Base URL and API Key inputs\n- Avoids Chrome/1Password autofill leaking saved login into config modal\n- Verified via Playwright MCP across multiple screen sizes and pages
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
…- Change modal container to flex-col, remove container overflow\n- Make content area flex-1 min-h-0 overflow-y-auto for reliable inner scroll\n- Keeps header visible and allows reaching bottom fields/actions
…dd overscroll-contain to modal content area\n- Improves wheel/trackpad behavior on desktop when content exceeds viewport
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
admin@admin.admin/adminadmininto the modal on the Results pageChanges
nameandautocompleteattributes and disable autocapitalize/spellcheck for Base URL and API Key inputsdata-lpignore,data-1p-ignore) to API Key fieldE2E Validation (local)
npm cithennpm run dev/and Results/resultsload without errorsNotes
next buildPull Request opened by Augment Code with guidance from the PR author