Skip to content

Integrate chat recovery, chronological history and shortcut help - #2830

Merged
Chris0Jeky merged 6 commits into
mainfrom
issue-2235/chat-shortcuts-integration
Sep 9, 2026
Merged

Integrate chat recovery, chronological history and shortcut help#2830
Chris0Jeky merged 6 commits into
mainfrom
issue-2235/chat-shortcuts-integration

Conversation

@Chris0Jeky

Copy link
Copy Markdown
Owner

This integration brings three independently reviewed milestone fixes onto the delivered Review/capture baseline: shortcut help follows feature availability and uses translated labels; chat keeps successful submissions visible through failed refreshes and provides explicit board-load retry; every chat consumer receives chronological history.

Preserves the commits from #2827 (aaa510b), #2828 (040c94d), and #2829 (fe4cf5c). Refs #1968, #2004, #2235. No parent issue completion claim.

Qualified candidate: 3923500 against 06bd4d1. Fifteen changed files, three conflict-free merge commits, no manual resolution edits.

Validation:

  • Full frontend: 393 files, 6243 passed / 3 skipped, exit 0.
  • Typecheck, build, nine-file scoped ESLint, diff check passed; doc links 674 files / 0 broken.
  • Chromium SC-005 passed using a fresh synthetic Mock database: link an unbound chat, explicitly continue its retained instruction, reach Review, and verify no board mutation.
  • Full backend: dotnet test backend/Taskdeck.sln -c Release -m:1: 9096 passed / 5 skipped, exit 0 (Domain1637, Application4245, API2907, CLI243, Architecture28, Integration36)..
  • Fresh independent Terra integration review: SHIP. All source heads and exact changed-file contents preserved; no hidden merge edits or causal HIGH/CRITICAL interaction found. Each source also had a fresh independent review.

Limits and follow-ups:

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 9, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-09T13:47:09.980325Z 3923500 PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@Chris0Jeky

Copy link
Copy Markdown
Owner Author

Coordinator cleanup: local and remote39235004b8f8ca7612d7f23969756afb30945e6b agreed, tracked tree clean, no worktree node/dotnet/testhost process remained. Ignored inventory was .tmp, backend bin/obj, frontend dist/node_modules and test-results. Copied all .tmp logs, synthetic Mock database and review diff OUT to .claude/local/milestone-20260909/preserved-2830/runtime-and-verification; copied test-results alongside it. Disposable generated/install trees required no preservation. Plain git worktree remove succeeded; remote branch/source commits preserved. Full backend session84117 completed exit0 with9096passed5skipped; logs preserved. Primary remains unchanged with its original two untracked directories. Hosted CI is pending; no merge claim.

@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: 39235004b8

ℹ️ 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 frontend/taskdeck-web/src/composables/useAutomationChat.ts
Comment thread frontend/taskdeck-web/src/components/chat/ChatMessageList.vue
@Chris0Jeky
Chris0Jeky merged commit cca0716 into main Sep 9, 2026
35 checks passed
@Chris0Jeky
Chris0Jeky deleted the issue-2235/chat-shortcuts-integration branch September 9, 2026 14:13
@github-project-automation github-project-automation Bot moved this from Review to Done in Taskdeck Execution Sep 9, 2026
@Chris0Jeky

Copy link
Copy Markdown
Owner Author

Delivered through #2830, merged as cca0716. Main tree c73859a9ba712192935fc13586260ab61a9fb22c exactly equals qualified39235004b; all three source heads are verified ancestors. GitHub automatically marked #2827/#2828/#2829 merged through preserved commits. Required ci-required.yml run34358736114 passed at the exact head/base, fresh Terra review passed and no unresolved threads remained. Tracked MEDIUMs remain follow-ups; no full parent-issue completion claim.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant