Skip to content

[codex] Improve Slack AI card status layout#213

Merged
LIU9293 merged 1 commit into
mainfrom
codex/slack-ai-card-status-layout
Jun 12, 2026
Merged

[codex] Improve Slack AI card status layout#213
LIU9293 merged 1 commit into
mainfrom
codex/slack-ai-card-status-layout

Conversation

@LIU9293

@LIU9293 LIU9293 commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Reshape Slack AI cards around the tested layout: title + agent + model + elapsed time, then stable rows for Run context, Current status, Tasks, and Tool calling.
  • Show build/plan mode and folder in Run context, and group task/tool details into stable secondary-list rows.
  • Update Slack streaming status cadence to one-second heartbeats for AI cards, and recreate a stale card if Slack reports message_not_in_streaming_state or streaming_state_conflict while appending.
  • Bump Ode version to 0.1.47.

Validation

  • bun run --bun tsc --noEmit
  • bun test packages/utils/test/status-stream.test.ts packages/core/test/runtime-resilience-e2e.test.ts
  • bun test
  • git diff --check

@LIU9293 LIU9293 marked this pull request as ready for review June 12, 2026 03:10

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 910e5be642

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread packages/core/kernel/request-run.ts Outdated
@LIU9293 LIU9293 force-pushed the codex/slack-ai-card-status-layout branch 2 times, most recently from 2c9b5cb to eaff307 Compare June 12, 2026 03:23

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: eaff307913

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread packages/core/kernel/request-run.ts
@LIU9293 LIU9293 force-pushed the codex/slack-ai-card-status-layout branch 2 times, most recently from 41c10f1 to 59e5d1a Compare June 12, 2026 03:30

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 59e5d1ab8b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread packages/core/kernel/request-run.ts
@LIU9293 LIU9293 force-pushed the codex/slack-ai-card-status-layout branch from 59e5d1a to 55ef602 Compare June 12, 2026 03:35

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 55ef602658

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread packages/core/kernel/request-run.ts Outdated
@LIU9293 LIU9293 force-pushed the codex/slack-ai-card-status-layout branch from 55ef602 to 275983c Compare June 12, 2026 03:48
@LIU9293 LIU9293 merged commit 3f4a83f into main Jun 12, 2026
2 checks passed

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 275983cc0e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +135 to +136
const elapsed = formatElapsedTime(startedAt);
return truncateField([title, state.agent?.trim(), state.model?.trim(), elapsed].filter(Boolean).join(" · "), 160);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Keep elapsed time out of the streaming diff key

When Slack status streaming is enabled, runOpenRequest drives this differ every 1s. Adding formatElapsedTime(startedAt) to the plan title makes planTitle !== lastPlanTitle on every tick because the formatted elapsed time includes seconds, so an otherwise idle card emits a plan_update and calls chat.appendStream every second. That burns about 60 appendStream requests/min per run and can hit the 100/min budget noted in this file with just two concurrent runs, causing rate-limit/fallback behavior even though no meaningful status changed.

Useful? React with 👍 / 👎.

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.

1 participant