Skip to content

[pull] main from vijaythecoder:main#4

Open
pull[bot] wants to merge 45 commits intoImViper:mainfrom
vijaythecoder:main
Open

[pull] main from vijaythecoder:main#4
pull[bot] wants to merge 45 commits intoImViper:mainfrom
vijaythecoder:main

Conversation

@pull
Copy link

@pull pull bot commented Aug 5, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.3)

Can you help keep this open source service alive? 💖 Please sponsor : )

- Add OpenAI Agents SDK dependencies (@openai/agents, @openai/agents-realtime) - Create new /realtime-agent-v2 route for parallel implementation - Implement MainV2.vue using SDK's RealtimeAgent and RealtimeSession - Migrate all 6 coaching tools to SDK's tool() format with Zod schemas - Update audio pipeline to use SDK's session.sendAudio() method - Add TypeScript declarations for SDK types - Maintain dual-agent architecture (salesperson + customer coach) - Use gpt-4o-mini models for cost optimization
- Extract 14 components from monolithic Main.vue (1558 lines) - Implement 3 Pinia stores for state management: - realtimeAgent: Business logic and conversation state - settings: UI preferences and toggles - openai: SDK agent management - Create modular component structure: - Navigation: TitleBar, CoachSelector, ConnectionStatus, etc. - Content: LiveTranscription, CustomerIntelligence, KeyInsights, etc. - Actions: CommitmentsList, PostCallActions - Modals: CustomerInfoModal - Integrate OpenAI Agents SDK while preserving exact UI - Maintain all 6 coaching function tools with SDK implementation - Preserve original 3-column grid layout and styling - Fix Tailwind CSS v4 @apply directive issues - Fix Zod nullable fields for OpenAI Realtime API compatibility - Update API endpoints to match existing routes
- Create comprehensive mock data service with realistic sales conversation - Add TypeScript interfaces for mock data structures - Implement mock mode toggle in navigation bar - Update store with mock data loading and simulation capabilities - Fix transcript styling: remove horizontal scroll, subtle borders - Include customer insights, coaching tips, and performance metrics - Enable easy UI testing without API calls
- Fix double scrollbar issue by using h-screen overflow-hidden pattern - Fix dropdown functionality with proper event handling - Fix template fetching to handle wrapped API response structure - Swap Discussion Topics and Post-Call Actions positions - Set column 3 component heights: Contextual 50%, Commitments 30%, Topics 20% - Fix overlay mode transparency by properly using useOverlayMode composable - Fix Customer Intelligence progress bar visibility in overlay mode - Fix Discussion Topics pill height to be fixed instead of stretching
- Fix hidden cards in single column view by adding minimum heights - Key Insights: min-h-[200px], Post-Call Actions: min-h-[150px] - Contextual Info: min-h-[250px], Commitments: min-h-[150px], Topics: min-h-[120px] - Update all card backgrounds to bg-gray-50/dark:bg-gray-900 for consistency - Simplify responsive layout to use only lg breakpoint for better mobile experience
- Adjust VAD settings for coach session: threshold 0.3, silence 100ms, padding 200ms - Add aggressive real-time analysis instructions to coach prompt - Fix duplicate transcript handlers - use input_audio_transcription.completed events - Add function call event handlers for coach analytics - Update conversation context on each customer message - Fix dropdown z-index with Teleport solution - Remove screen recording blocked indicator - Add debug logging for function calls
The architecture documentation has been moved to a more appropriate location or is no longer needed.
- TitleBar.vue: Add ESLint comment for false positive on props usage - MainV2.vue: Remove unused imports (RealtimeSessionOptions, ref) and unused variables (currentAudioData, calculateAudioLevel) - openai.ts: Remove unused RealtimeAgentConfiguration import - openai-agents.d.ts: Replace Function type with proper function signatures All linting issues resolved - npm run lint now passes successfully
- Remove screen darkening effect from protection mode - Update card backgrounds to white for better contrast in light theme - Remove shadows from transcription message bubbles - Update system messages to professional gray theme - Fix WebSocket connection errors by correcting parameter names - Implement smart transcript grouping to combine messages from same speaker - Fix tool configuration format for coach session - Add 5-second time window for message grouping These changes improve the visual appearance and functionality of the realtime agent interface.
- Update AppSidebar logo link to /realtime-agent-v2 - Update Conversations page link to /realtime-agent-v2 - Update Onboarding page link to /realtime-agent-v2 All navigation now points to the new V2 implementation instead of the legacy version.
- Replace Swift-based system audio capture with electron-audio-loopback - Add microphone permission handling for macOS - Fix screen recording permission loop issue - Add NSScreenCaptureUsageDescription to Info.plist - Create Audio Test page for testing mic and system audio - Add navigation links between Audio Test, Agent V1, and Agent V2 - Update MainV2 to use electron-audio-loopback for system audio - Add microphone entitlement com.apple.security.device.audio-input - Create API endpoints for checking and requesting media permissions - Add comprehensive troubleshooting documentation
- Remove V2 badge from MainV2.vue - Remove Audio Test and Agent V2 links from navigation - Remove Agent V1 link for cleaner navigation - Remove Mock Mode button from UI - Add console commands for mock mode (window.clueless.enableMockMode/disableMockMode) - Improve connection status display with better visual representation - Show 'Connected' in green, 'Connecting...' in yellow with animation - Use 'Not Connected' instead of 'disconnected' - Add color-coded text and smooth transitions - Simplify developer tools to essential commands only
- Remove debug console.log statements throughout the component - Keep only critical error messages and error handling logs - Remove audio capture status checks and session configuration logs - Remove transcription completion logs and transport event logs - Remove permission status logs and audio sending debug logs - Clean up mock mode console messages - Remove developer commands listing on startup - Maintain clean console output for production use
- Fix missing closing brace in onaudioprocess callback - Remove unnecessary nested try-catch block for audio loopback - Ensure all async/await code is properly contained within async functions - Build now completes successfully
- Remove unused closeMobileMenu function from MobileMenu.vue - Remove unused parameters from catch blocks in MainV2.vue - Remove unused event parameters from event handlers - Use parameterless catch blocks where error object is not needed - All ESLint checks now pass
- Remove 19 debug logs from useScreenProtection.ts - Remove 50+ debug logs from RealtimeAgent/Main.vue - Remove 6 debug logs from AudioTest/Index.vue - Remove 10 debug logs from audioCapture.ts - Remove 4 debug logs from electronPermissions.ts - Remove 1 debug log from MainV2.vue - Fix ESLint errors from orphaned console.log arguments - Keep legitimate error handling console.error statements
- Replace window.audioLoopback with Native.ipcRendererInvoke - Use dynamic IPC channel names: enable-loopback-audio, disable-loopback-audio - Consistent with new NativePHP extension system
- Fixed electron-audio-loopback integration in NativePHP extension - Removed duplicate IPC handler registration that was causing conflicts - Updated MainV2.vue to use Native.ipcRendererInvoke for audio loopback - Added comprehensive logging for debugging audio capture flow - Added nativephp-electron to .gitignore for local development setup The key fix was allowing the electron-audio-loopback package to register its own IPC handlers without interference from manual implementations.
- Add node-mac-permissions dependency to NativePHP package - Create permissions.js module with IPC handlers for camera, microphone, screen, and folder permissions - Fix native module bundling issue by externalizing node-mac-permissions in vite config - Update API calls to use correct node-mac-permissions methods (getAuthStatus, askForMicrophoneAccess, etc.) - Add contextBridge API exposure in preload script for secure renderer access - Enable contextIsolation and integrate permission buttons in TitleBar.vue - Remove redundant macPermissions.ts utility file - Clean up .DS_Store files and build artifacts
…anch - Switch from local path to GitHub repository - Use feature/macos-permissions-enhanced branch with permissions support - Update composer.lock with latest dependencies
- Fixed 'permissions is not defined' error in MainV2.vue by declaring variable outside try block - Added defensive checks for permissions object properties - Updated composer.json to use local NativePHP package via symlink - Cleaned up console logs and debug statements - Added proper error handling for permission checks
…on compatibility - Update composer.json to use local nativephp-electron package - Modify useScreenProtection composable to use new IPC-based approach - Add support for window.macPermissions.screenProtection API - Maintain fallback to window.remote for backward compatibility - Fix 'N/A' display issue when context isolation is enabled
- Implement overlay mode IPC handlers in permissions.js - Update preload script to expose overlay mode methods via contextBridge - Refactor useOverlayMode composable to use IPC as primary method - Fix settings store initialization in Main.vue - Maintain backward compatibility with window.remote fallback - Enable transparency and window opacity controls via secure IPC
- Remove console.log statements for permission checks when successful - Hide mic/screen permission buttons in TitleBar when permissions are granted - Remove success messages for permissions and dual audio capture - Update call started message to be more concise - Only show logs and UI elements when there's a problem - Update composer.json to use GitHub repository for nativephp/electron
- Remove customer info modal - calls now start immediately - Change 'Call History' to 'Dashboard' in navigation - Fix all ESLint errors in nativephp-extension.js - Fix PHP code style issues with Laravel Pint - Remove local nativephp-electron folder (now using GitHub package) - All tests passing successfully
…form CI - Moved node-mac-permissions from dependencies to optionalDependencies - Updated GitHub Actions workflows to use npm ci --omit=optional - Fixes EBADPLATFORM error on Linux CI runners - Package still installs on macOS but is skipped on other platforms
- Remove CheckOnboarding middleware and its test - Delete standalone Onboarding.vue page component - Add new OnboardingModal.vue component for inline onboarding - Update MainV2.vue to integrate onboarding modal - Remove onboarding route from web.php - Simplify bootstrap/app.php by removing onboarding middleware This change improves UX by keeping users in context during onboarding rather than redirecting to a separate page.
- Add public/hot to cleanup_exclude_files to prevent Vite dev mode in production - Add npm install step before npm run build in prebuild commands - Create build-prepare.sh script for clean production builds - Add comprehensive build fix documentation This ensures frontend assets are properly built and included when distributing the NativePHP application, fixing "Vite not found" errors on fresh installations.
vijaythecoder and others added 13 commits August 2, 2025 20:17
- Changed NativeAppServiceProvider to check for existing system templates instead of user count - Updated MeetingTemplateSeeder to use firstOrCreate() to prevent duplicates - Ensures templates are only seeded once on first app launch 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Added session tracking and management - Create conversation session when call starts - Implement periodic saving every 5 seconds - Save all data when call ends - Add watchers to automatically queue transcripts - Queue insights, topics, commitments, and action items - Ensure all conversation data is persisted to database This restores the call history functionality that was working in v1. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
…ies bug - Remove package-lock.json and node_modules before npm install - Use --no-optional flag to avoid platform-specific dependencies - Remove npm cache from setup-node action since we delete package-lock.json - Fixes build failures on Linux CI environments
- Document all major features and improvements from PR #25 - Highlight first stable release milestone - Include migration guide for users upgrading from pre-releases
…ntegration Feature/macos permissions integration
- Update actions/upload-artifact from v3 to v4 - Update actions/download-artifact from v3 to v4 - Update actions/cache from v3 to v4 - Replace deprecated create-release and upload-release-asset with gh CLI - Fix automatic release asset uploads for v1.0.0
- Update PHP version from 8.2 to 8.3 to match NativePHP requirements - Remove npm cache since we delete package-lock.json - Apply npm optional dependencies workaround in build process
- Remove incorrect --x64/--arm64 flags - Use architecture as positional argument in native:build command
- Add write permissions to release workflow for uploading assets - Create export-certificate.sh helper script for certificate preparation - Document NativePHP build variables in .env.example - Prepare for automated code signing in GitHub Actions
- Add note about Apple Developer ID signing - Trigger rebuild with proper code signing certificates
- Allow workflow to run on documentation changes - Enable rebuild for signed release assets
- Add verify-certificate.sh to test certificate password - Add export-certificate-simple.sh for manual export method - Help troubleshoot certificate password issues
@pull pull bot locked and limited conversation to collaborators Aug 5, 2025
@pull pull bot added the ⤵️ pull label Aug 5, 2025
krishnakishorep and others added 2 commits January 8, 2026 14:50
…iables - Add environment variables for code signing and notarization in release workflow - Include new permission for Bash find command in settings.local.json
feat: enhance release workflow with notarization and code signing var…
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

2 participants