Skip to content

fix(web): bound asynchronous attachment intake - #133

Closed
oleksoleksoleks wants to merge 3 commits into
LycaonLLC:mainfrom
oleksoleksoleks:fix/pr-122-attachment-intake
Closed

fix(web): bound asynchronous attachment intake#133
oleksoleksoleks wants to merge 3 commits into
LycaonLLC:mainfrom
oleksoleksoleks:fix/pr-122-attachment-intake

Conversation

@oleksoleksoleks

Copy link
Copy Markdown
Contributor

Summary

Supersedes #122.

Screenshots

No visual change.

Testing

  • pnpm --filter @t4-code/web typecheck
  • pnpm --filter @t4-code/web test (86 files, 1219 tests passed)

Risk

  • Later picker batches wait for earlier materialization instead of reading concurrently.

Notes

  • Rebased on current main.

AI Review Report

  • Resolves session resurrection and concurrent Android WebView memory-bound findings.

Security Audit

  • Prevents stale async work from recreating deleted session state.

@oleksoleksoleks
oleksoleksoleks force-pushed the fix/pr-122-attachment-intake branch from 5005590 to 4bbb427 Compare July 21, 2026 16:11
@oleksoleksoleks

Copy link
Copy Markdown
Contributor Author

Closing: #122 merged with a stronger reservation/token implementation that already addresses both requested lifecycle and concurrent-memory blockers. This replacement is now redundant.

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