Skip to content

fix: add missing closing brace for view(resolver:) in Screen.swift#32

Draft
Sjoerd-Bo3 with Copilot wants to merge 29 commits into
devfrom
copilot/add-settings-audit-log
Draft

fix: add missing closing brace for view(resolver:) in Screen.swift#32
Sjoerd-Bo3 with Copilot wants to merge 29 commits into
devfrom
copilot/add-settings-audit-log

Conversation

Copilot AI commented Apr 7, 2026

Copy link
Copy Markdown
  • Remove duplicate ShareSheet struct (previous fix)
  • Replace computed properties with stored state to prevent body-render crash (previous fix)
  • Consolidate two .sheet modifiers into a single sheet(item:) with an ActiveSheet enum to prevent UIActivityViewController/SwiftUI sheet conflicts
  • Add onDismiss cleanup for pending note saves
  • Validation passed (3 minor pre-existing style suggestions, no issues in our changes)

Copilot AI and others added 5 commits April 7, 2026 08:57
- Add SettingsChangeStored Core Data entity with fetch indexes
- Add SettingsChangeStored CoreData class and properties files
- Add SettingsAuditStorage protocol and BaseSettingsAuditStorage implementation
- Register SettingsAuditStorage in StorageAssembly
- Add SettingsMetadataRegistry mapping setting keys to human-readable metadata
- Integrate audit logging in BaseSettingsManager for TrioSettings and Preferences changes
- Add audit logging to BasalProfileEditor, ISFEditor, CarbRatioEditor, TargetsEditor providers
- Add SettingsAuditLog module (DataFlow, Provider, StateModel, RootView)
- Add settingsAuditLog screen case to Router
- Add 'Settings Change Log' entry point in SettingsRootView

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: Sjoerd-Bo3 <2100083+Sjoerd-Bo3@users.noreply.github.com>
Use a single DateFormatter for both grouping and sorting to avoid
repeated instantiation inside the sort closure.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: Sjoerd-Bo3 <2100083+Sjoerd-Bo3@users.noreply.github.com>
All DateFormatter instances in SettingsAuditLog views and StateModel are
now static properties, created once and reused across calls and rows.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: Sjoerd-Bo3 <2100083+Sjoerd-Bo3@users.noreply.github.com>
Copilot AI and others added 3 commits April 7, 2026 09:53
…Phase and fix indentation

Agent-Logs-Url: https://github.com/Sjoerd-Bo3/Trio/sessions/3ec37fd3-d9a8-454c-a053-287c2b5da009

Co-authored-by: Sjoerd-Bo3 <2100083+Sjoerd-Bo3@users.noreply.github.com>
…or-based diff

Agent-Logs-Url: https://github.com/Sjoerd-Bo3/Trio/sessions/3ec37fd3-d9a8-454c-a053-287c2b5da009

Co-authored-by: Sjoerd-Bo3 <2100083+Sjoerd-Bo3@users.noreply.github.com>
…sChangeStored files

Agent-Logs-Url: https://github.com/Sjoerd-Bo3/Trio/sessions/8e054e2e-c2c5-4e1e-81c1-bd02b42a2f15

Co-authored-by: Sjoerd-Bo3 <2100083+Sjoerd-Bo3@users.noreply.github.com>
Copilot AI and others added 2 commits April 7, 2026 10:38
… and track cgmPluginIdentifier

Agent-Logs-Url: https://github.com/Sjoerd-Bo3/Trio/sessions/74b2250e-7323-4364-b102-bfa47164b7cf

Co-authored-by: Sjoerd-Bo3 <2100083+Sjoerd-Bo3@users.noreply.github.com>
Copilot AI and others added 2 commits April 7, 2026 11:13
…d of Core Data managed objects in UI

Agent-Logs-Url: https://github.com/Sjoerd-Bo3/Trio/sessions/de91fd5c-b81a-418a-8fd6-cd718d2c177d

Co-authored-by: Sjoerd-Bo3 <2100083+Sjoerd-Bo3@users.noreply.github.com>
Copilot AI and others added 3 commits April 7, 2026 11:41
…ebounceTask on disappear and adding @mainactor to mutation methods

Agent-Logs-Url: https://github.com/Sjoerd-Bo3/Trio/sessions/76049541-2cf7-4f68-9669-ffce49338803

Co-authored-by: Sjoerd-Bo3 <2100083+Sjoerd-Bo3@users.noreply.github.com>
…ror while keeping .onDisappear crash fix

Agent-Logs-Url: https://github.com/Sjoerd-Bo3/Trio/sessions/ff21c582-932d-42b2-918b-adef0fa9d374

Co-authored-by: Sjoerd-Bo3 <2100083+Sjoerd-Bo3@users.noreply.github.com>
- convert core data managed objects to value types atomically inside
  performAndWait via new fetchChangeEntries method, preventing
  faulting/invalidation between fetch and property access
- add explicit @mainactor isolation to debounce task to prevent
  concurrent entries array mutation during view teardown
- flush pending note save synchronously on disappear instead of
  just cancelling the debounce task
- add Sendable conformance to ChangeEntry for compile-time safety
Copilot AI and others added 2 commits April 7, 2026 13:05
… prevent crash

Agent-Logs-Url: https://github.com/Sjoerd-Bo3/Trio/sessions/14755bc8-a3e8-4e17-87a0-9db1229e996e

Co-authored-by: Sjoerd-Bo3 <2100083+Sjoerd-Bo3@users.noreply.github.com>
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.

3 participants