Skip to content

Refine AI Providers, History and Stats - #955

Open
altic-dev wants to merge 11 commits into
mainfrom
B/1.6.10-more-enhancements
Open

Refine AI Providers, History and Stats#955
altic-dev wants to merge 11 commits into
mainfrom
B/1.6.10-more-enhancements

Conversation

@altic-dev

@altic-dev altic-dev commented Sep 9, 2026

Copy link
Copy Markdown
Owner

Description

Refresh AI Providers, History and Stats; improve clipboard handling and prompt-test feedback.

Type of Change

  • 🐞 Bug fix
  • ✨ New feature

Related Issue or Discussion

Follow-up to #950.

Testing

  • Tested on Intel Mac
  • Tested on Apple Silicon Mac
  • Tested on macOS version: 27
  • Ran linter locally: swiftlint --strict --config .swiftlint.yml Sources Tests Package.swift
  • Ran formatter locally: swiftformat --config .swiftformat Sources
  • Ran tests locally: 8 suites + 2 XCTest cases.

Screenshots / Video

image image

Notes

Redesign provider management and Fluid Intelligence model browsing with isolated setup and operation state. Persist optional per-model verification and clarify default-provider readiness.

Preserve clipboard contents safely, add regression coverage and design concepts, and align beta version metadata.
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 9, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-10T03:12:17.988484Z 38730e9 New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown

The PR Policy check is blocking this PR because required template information is missing.

Please update the PR description with:

  • Screenshots / Video

Visual files detected:

  • Sources/Fluid/ContentView.swift
  • Sources/Fluid/Persistence/SettingsStore.swift
  • Sources/Fluid/Theme/AppTheme.swift
  • Sources/Fluid/Theme/Components/FluidGlassControls.swift
  • Sources/Fluid/Theme/Components/FluidManagementSheet.swift
  • Sources/Fluid/Theme/Components/FluidModelUpdateButton.swift
  • Sources/Fluid/Theme/Components/OnboardingComponents.swift
  • Sources/Fluid/UI/AISettings/AIEnhancementSettingsView.swift
  • Sources/Fluid/UI/AISettings/AIEnhancementSettingsViewModel+ProviderSetup.swift
  • Sources/Fluid/UI/AISettings/AIEnhancementSettingsViewModel.swift
  • Sources/Fluid/UI/AISettings/AIProvidersDesignPreview.swift
  • Sources/Fluid/UI/AISettings/AISettingsScreens.swift
  • Sources/Fluid/UI/AISettings/AddProviderSheet.swift
  • Sources/Fluid/UI/AISettings/DictationDefaultProvider.swift
  • Sources/Fluid/UI/AISettings/FluidIntelligenceLiveSection.swift
  • Sources/Fluid/UI/AISettings/FluidIntelligenceModelCarousel.swift
  • Sources/Fluid/UI/AISettings/FluidModelShowcaseCard.swift
  • Sources/Fluid/UI/AISettings/PrivateAIModelPresentation.swift
  • Sources/Fluid/UI/AISettings/PrivateAISettingsController.swift
  • Sources/Fluid/UI/AISettings/PrivateAISettingsSession.swift
  • Sources/Fluid/UI/AISettings/ProviderModelVerificationStore.swift
  • Sources/Fluid/UI/AISettings/ProviderSetupDraft.swift
  • Sources/Fluid/UI/AISettingsView+AIConfiguration.swift
  • Sources/Fluid/UI/AISettingsView+AdvancedSettings.swift
  • Sources/Fluid/UI/AISettingsView.swift
  • Sources/Fluid/UI/HistoryAppIcon.swift
  • Sources/Fluid/UI/HistoryInlineAudioView.swift
  • Sources/Fluid/UI/HistoryTextComparisonView.swift
  • Sources/Fluid/UI/HistoryTextDiff.swift
  • Sources/Fluid/UI/ModelDisplayName.swift
  • Sources/Fluid/UI/SearchableModelPicker.swift
  • Sources/Fluid/UI/StatsSnapshot.swift
  • Sources/Fluid/UI/StatsSnapshotStore.swift
  • Sources/Fluid/UI/StatsView.swift
  • Sources/Fluid/UI/TranscriptionHistoryView.swift
  • Tests/PrivateAISettingsBoundaryTests.swift
  • Tests/PrivateAISettingsControllerDoubles.swift
  • scripts/test_private_ai_settings_boundary.sh

Screenshots or video are required for UI, UX, settings, onboarding, overlay, menu bar, or visual behavior changes. If this PR has no visual changes, check the no-visual-change box in the template.

If this remains incomplete for 48 hours after opening, the PR may be closed.

@github-actions github-actions Bot added needs screenshots Pull request needs screenshot or video evidence. and removed needs screenshots Pull request needs screenshot or video evidence. labels Sep 9, 2026
@greptile-apps

greptile-apps Bot commented Sep 9, 2026

Copy link
Copy Markdown

Greptile Summary

Refines provider-management persistence and strengthens media restoration after transient Now Playing metadata failures.

  • Avoids unnecessary Keychain writes when a managed provider key is unchanged.
  • Retains verified media-pause ownership across bounded metadata outages so a later lifecycle event can safely retry restoration.
  • Adds regression coverage for transient and prolonged media-query failures.

Confidence Score: 5/5

The changes since the previous review appear safe to merge, with no accepted new findings.

The provider-key optimization preserves changed-key persistence, and media restoration now retains ownership without issuing an unsafe blind Play command. altic-dev accepted the clipboard timing risk because the bounded 0.5-second restoration behavior is preferred until real-world wrong-paste evidence emerges. The missing-audio thread and media-pause thread were manually resolved without explanation.

Reviews (6): Last reviewed commit: "Retain media recovery ownership and skip..." | Re-trigger Greptile

Comment thread Sources/Fluid/Services/TypingService.swift
Comment thread Sources/Fluid/UI/TranscriptionHistoryView.swift Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 16cabf7323

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread Sources/Fluid/Services/TypingService.swift
@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown

FluidVoice PR build ready

Download FluidVoice-PR-955-38730e98cef2

The artifact contains the ad-hoc-signed app ZIP, Xcode archive, build manifest, and installation instructions. It expires 5 days after the build.

Install the app

  1. Extract the downloaded artifact, then extract FluidVoice-PR-955.app.zip.

  2. Move FluidVoice Refine AI Providers, History and Stats #955.app into the /Applications folder.

  3. Open Terminal and remove the download quarantine marker:

    xattr -dr com.apple.quarantine "/Applications/FluidVoice #955.app"
    
  4. In Applications, Control-click FluidVoice Refine AI Providers, History and Stats #955.app and choose Open.

  5. If macOS still blocks it, open System Settings → Privacy & Security, click Open Anyway, and confirm.

This build has its own app identity, so its permissions are separate from the release version of FluidVoice.

View workflow run

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: cf3005e631

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread Sources/Fluid/UI/AISettingsView+AIConfiguration.swift
Comment thread Sources/Fluid/UI/TranscriptionHistoryView.swift Outdated
* fix media playback coordination during dictation

* retry transient media resume queries

* bound playback recovery and quit cleanup

* retry transient initial playback checks
Comment thread Sources/Fluid/Services/MediaPlaybackService.swift

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9db0631a15

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread Sources/Fluid/UI/AISettings/AIEnhancementSettingsViewModel.swift Outdated
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.

1 participant