fix(web): avoid false draft attachment persistence warnings

[shivamhwp]

Summary

• flush pending composer draft writes before verifying persisted image attachments
• prevent draft attachments from being marked as unsaved during the debounce window
• add a regression test covering attachment verification after the debounced write is flushed

Fixes #979

Before

After

Why

The composer draft store debounces localStorage writes, but attachment verification was reading storage immediately in a microtask. That could race the pending write and incorrectly flag persisted attachments as non-persisted, which surfaced the "Draft attachment could not be saved locally" warning even when persistence would have succeeded.

Test plan

• bun fmt
• bun lint
• bun typecheck

UI evidence

• This is a small UI behavior fix with no visual layout change.
• The warning now only appears for real persistence failures instead of debounce timing.

Made with Cursor

Note

[!NOTE]

Fix false draft attachment persistence warnings in composer draft store

• Extracts a verifyPersistedAttachments helper in composerDraftStore.ts that flushes pending debounced storage writes before reading persisted attachment IDs, preventing race conditions where attachments appeared unpersisted.
• Storage read errors are now caught and treated as an empty persisted ID set instead of throwing.
• Adds a test in composerDraftStore.test.ts that verifies nonPersistedImageIds is empty after the debounced write is flushed.

^{Macroscope summarized 5aa4f85.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 9b475000-ae82-4b69-bdf9-fa3b39ebfc59

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

📝 Coding Plan

• Generate coding plan for human review comments

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Tip

CodeRabbit can use Trivy to scan for security misconfigurations and secrets in Infrastructure as Code files.

Add a .trivyignore file to your project to customize which findings Trivy reports.