fix(mobile): share request action button defaults

[juliusmarminge]

Targets main independently. This PR has no dependency on V2 or another UI audit PR.

Approval, task-creation failure, and answer-submission cards hand-styled equivalent actions. RequestActionButton now owns button semantics, size, theme colors, and pressed/disabled feedback. Action handlers remain with the cards; edit-task uses the semantic primary color.

Validation: pendingUserInputLayout and pending-thread-creation tests, affected client typecheck, scoped lint, and React Doctor review. React Doctor’s existing consumer findings remain documented from the original audit; no new component findings were identified.

Code-only verification was requested; no browser/device pass or before/after screenshots are included. No wire contracts, persisted data, or upgrade migrations change.

Review status: ready for review and unmerged. Macroscope requires human review of the theme-based primary styling and pressed/disabled defaults.

Model: GPT-6. Harness: Codex.

Summary by CodeRabbit

• UI Improvements
  • Standardized action buttons across thread cards for pending approvals, user input, and failed thread creation.
  • Preserved existing labels, disabled states, styling tones, and button actions.
  • Improved consistency of button sizing, colors, and accessibility metadata across these actions.

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.5 KiB	13.5 KiB	+5 B (+0.0%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.1 KiB	7.1 KiB	+1 B (+0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.5 KiB	6.5 KiB	+4 B (+0.1%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	56.3 KiB	56.3 KiB	0 B (0.0%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.5 KiB	13.5 KiB	−5 B (−0.0%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.1 KiB	7.1 KiB	+4 B (+0.1%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.4 KiB	6.4 KiB	−9 B (−0.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	57.0 KiB	57.0 KiB	0 B (0.0%)	66.4 KiB	✅
Claude	Live turn messages	9	9	0 (0.0%)	21	✅

Baseline: 53510d4 · PR result: 20e268f · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 113.9 KiB
• Claude decoded thread snapshot: 114.6 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This PR centralizes request-card buttons and changes product-facing defaults such as theme-based primary styling, active feedback, and disabled presentation. The affected behavior is small and localized, but the default changes warrant human review.

^{You can add or adjust custom eligibility rules. Learn more.}

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: efab65f8-c13a-49e5-b2f4-ce17e8c4734b

📥 Commits

Reviewing files that changed from the base of the PR and between 53510d4 and 20e268f.

📒 Files selected for processing (4)

• apps/mobile/src/features/threads/PendingApprovalCard.tsx
• apps/mobile/src/features/threads/PendingUserInputCard.tsx
• apps/mobile/src/features/threads/RequestActionButton.tsx
• apps/mobile/src/features/threads/ThreadCreationFailedCard.tsx

Limit details: You’ve used all 10 included reviews currently available.

---

📝 Walkthrough

Walkthrough

A shared RequestActionButton component now provides consistent thread-card action styling and accessibility behavior. Approval, user-input, and thread-creation-failure cards use it while preserving their existing labels, callbacks, and disabled conditions.

Changes

Thread action buttons

Layer / File(s)	Summary
Shared action button component apps/mobile/src/features/threads/RequestActionButton.tsx	Added RequestActionButton with configurable labels, tones, sizes, disabled accessibility state, styles, and forwarded Pressable props.
Thread card adoption apps/mobile/src/features/threads/PendingApprovalCard.tsx, apps/mobile/src/features/threads/PendingUserInputCard.tsx, apps/mobile/src/features/threads/ThreadCreationFailedCard.tsx	Replaced card-specific Pressable elements with RequestActionButton. Existing labels, callbacks, keys, disabled conditions, and action tones remain wired.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~15 minutes

Change: Bug fix

Suggested reviewers: t3dotgg

Merge Risk: ⚪ Minimal · up to 20e26

The shared button adoption preserves the existing action wiring and disabled conditions in the reported integrations. No actionable merge risk remains.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 4 functions across 4 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly identifies the main change: sharing default behavior through the mobile request action button.
Description check	✅ Passed	The description explains the button refactor, rationale, validation, scope, and the absence of screenshots. It does not use the template headings or complete the checklist, but it provides the require…

• Fix all pre-merge checks with AI

✨ Finishing Touches 💡 1

📝 Generate docstrings 💡

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

---

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