Skip to content

feat(mobile): show delegated work and retry actions - #327

Open
leoisadev1 wants to merge 20 commits into
stack/22-work-cardsfrom
stack/23-mobile-work-cards
Open

leoisadev1 wants to merge 20 commits into
stack/22-work-cardsfrom
stack/23-mobile-work-cards

Conversation

@leoisadev1

@leoisadev1 leoisadev1 commented Sep 28, 2026 •

Copy link
Copy Markdown
Member

Part 23 of 29. Stacked on #326.

Mobile chats needed the same visible delegated work and reverse actions as web. This cut adds mobile work cards, result and speaker labels, retry-once and routine delegation actions for LEO-607, LEO-608, LEO-609, LEO-610, LEO-611, LEO-612 and LEO-613. Child work stays on cards instead of appearing as ordinary chats; channel failures keep their reason and assignment.

The cards use upstream #300's cached feed and compare offered actions so an earlier card updates after retry. Web rows retain #299 memoization. A frozen install and all five package typechecks passed with zero errors and warnings. No physical phone was tested.

Screenshots

Captures use fictional fixture data on web at 1440x900.

Failed delegated work now offers a Try again action beside the Mira work card.

Before After
Failed work before Failed work after

Created with Claude Opus 5.5 and GPT-6 Sol in T3 Code.


Devin Review

@vercel

vercel Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
akeru-bot-landing Ready Ready Preview Sep 28, 2026 5:40pm UTC

Request Review

@github-actions github-actions Bot added type:provider Agent provider contribution. area:directory Plugin directory and its user interface. area:connectors Plugin and MCP connector runtime. size:XXL vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. labels Sep 28, 2026
devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

@greptile-apps

greptile-apps Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

No outstanding finding blocks merging.

Summary

This PR adds mobile delegated-work cards and retry actions, with supporting delegation, routine, channel, and provider updates. No new actionable issue was identified. The five previous findings are resolved and are not reposted.

Reviews (42) · Last reviewed commit: "fix(web): channel repair links select th..."

greptile-apps[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

greptile-apps[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

@devin-ai-integration devin-ai-integration 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.

Note

Newer findings are available below. Devin Review posted a newer report on this PR, in addition to the findings presented here.

Devin Review found 2 new potential issues.

Devin Review

Comment on lines +878 to +895
{
botId: original.childBotId,
task: original.task,
expectedResult: original.expectedResult,
...(deadline ? { deadline } : {}),
allowedToolIds: original.access.allowedToolIds,
memoryScopes: original.access.memoryScopes,
mcpServerIds: original.access.enabledMcpServerIds,
sandbox: original.access.sandbox,
runtimeMode: original.access.runtimeMode,
approvalCeiling: original.access.approvalCeiling,
keep: original.keep,
},
{
trigger: original.trigger,
retryOfDelegationId: original.delegationId,
anchorMessageId: original.anchorMessageId,
},

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔍 Retries omit contextual background

A retry reuses the stored task and access grant, but not the original context, which the record does not retain. The docs disclose this limit; review whether the action needs an in-product warning for context-dependent work.

Devin Review


Was this helpful? React with 👍 or 👎 to provide feedback.

Comment on lines +323 to +325
A delegation that ends while the server is down, or before `routine.run.start`
records the run's `threadRef`, does not settle the run. Recovering those runs after a restart is
follow-up work.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔍 Documented gap in scheduled recovery

The new scheduled-work path has no restart reconciliation for a delegation that finishes while the server is down. The internal docs acknowledge this gap; track it before relying on routine status across restarts.

Devin Review


Was this helpful? React with 👍 or 👎 to provide feedback.

leoisadev1 and others added 20 commits September 29, 2026 13:35
…r matrix (LEO-609, LEO-610, LEO-612)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…firm

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…le card actions (LEO-611, LEO-613)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Compare the offered card actions when memoized feed rows decide whether to render. Cover the retry case where a later delegation changes the earlier card without changing its record.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Apply the repository formatter to files changed on stack branch 23. Add an empty changeset for this CI-only formatting fix.

GPT-6 Sol via T3 Code Codex harness.
The decider refuses to cancel a run that already ended, and a late cancel
from settled bot work leaves that outcome in place. Canceled scheduled work
settles its claim as canceled, including when the user cancels the run. A run
whose start fails for any reason other than having ended is blocked with that
reason and its bot work is canceled.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Scheduled work now gets its own turn id instead of borrowing the chat's
current turn, so later replies in that turn no longer pull the card down.
Without an anchor message, a scheduled card sits at its creation time rather
than the end of the chat.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
When a new delegation cannot record its running state or start its child
turn, the record now fails with the reason instead of staying open with no
watcher, so Try again reappears. The runtime also refuses to retry work that
already has a successor, covering two retries accepted before either started.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
plainText now removes single-star and underscore emphasis without touching
snake_case words, and turns star or plus list bullets into plain dashes.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
A remembered chat path without a thread shell no longer enables routine
creation that would fail on submit.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The chat guide still called mobile work cards read-only after they gained
Let it finish, Cancel, and Try again.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Settings now keeps the selected channel tab in its store rather than
reading the URL hash, so the roster sheet's update-credentials action sets
that tab before it opens Settings.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

@devin-ai-integration devin-ai-integration 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.

Devin Review found 1 new potential issue.

Devin Review

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔍 Channel recovery lacks user guidance

A channel can now recover in the attempted project after reassignment fails. The repository requires user-visible behavior changes in user documentation; this fallback is undocumented.

(Refers to this code)

Devin Review


Was this helpful? React with 👍 or 👎 to provide feedback.

This branch was successfully deployed

1 active (outdated) deployment
Preview — 90a7454d Deployed Sep 29, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:connectors Plugin and MCP connector runtime. area:directory Plugin directory and its user interface. size:XXL type:provider Agent provider contribution. vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant