Skip to content

Clarify bridge previews and damaged source-link diagnostics - #19

Merged
TheLinuxGuy merged 1 commit into
codex/iphone-tasks-compatibilityfrom
fix/bridge-plan-counts
Oct 26, 2026
Merged

TheLinuxGuy merged 1 commit into
codex/iphone-tasks-compatibilityfrom
fix/bridge-plan-counts

Conversation

@TheLinuxGuy

Copy link
Copy Markdown
Owner

Problem and behavior

Real-iPhone acceptance for #17 found that plan called metadata-only changes renames (F-3), and damaged or removed source links could produce misleading row-reordering/context messages (F-5). plan now counts description renames separately and reports fields, recurrence and deletion. Rejections identify trailing prose, a missing/malformed link, or a changed identity while preserving the original rejection/recovery behavior.

The real rejected iPhone save at 2026-10-26 09:37 UTC contained curly quotes around the fw: identity and APPENDED-AFTER-LINK after the link. The 09:42 UTC save removed C05's source link. Five new real-CLI checks cover those shapes, a corrupted identity, metadata preview counts and preview immutability. No task text is echoed in the status report.

F-3 also exists on main. A read-only main diagnostic rejects the two link examples with its older generic source-link instruction; the misleading structural messages occur in #17's stable-row parser.

Validation

  • Baseline: 824/824 self-tests.
  • New regressions fail before the fixes; final full self-test: 829/829 passed.
  • Public documentation: 248/248 passed; Ruff: passed.
  • Production coverage: 93.89% (85% gate); 15/15 changed executable production lines covered (100%, above 90%).
  • Device and scheduled-job re-verification is continuing in the isolated fictional-data vault. Acceptance evidence stays outside this public repository. This PR does not certify release readiness.

Based on #17's f9426e2. No release, tag, or merge is requested.

Distinguish description renames from field updates in plan, and diagnose trailing prose or damaged generated source links before structural checks.

Agent: Codex
Model: gpt-6
Copilot AI balanced review requested due to automatic review settings October 26, 2026 16:40

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

@codecov

codecov Bot commented Oct 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (codex/iphone-tasks-compatibility@f9426e2). Learn more about missing BASE report.

Additional details and impacted files
@@                         Coverage Diff                         @@
##             codex/iphone-tasks-compatibility      #19   +/-   ##
===================================================================
  Coverage                                    ?   93.03%           
===================================================================
  Files                                       ?       18           
  Lines                                       ?     3734           
  Branches                                    ?        0           
===================================================================
  Hits                                        ?     3474           
  Misses                                      ?      260           
  Partials                                    ?        0           
Files with missing lines Coverage Δ
_tools/icloud_bridge.py 94.22% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@TheLinuxGuy
TheLinuxGuy merged commit d384906 into codex/iphone-tasks-compatibility Oct 26, 2026
13 checks passed
@TheLinuxGuy
TheLinuxGuy deleted the fix/bridge-plan-counts branch October 26, 2026 17:21
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.

2 participants