Skip to content

[codex] Add Codex thread links and Linear comment resumes#84

Merged
yansenzhou-oai merged 1 commit into
mainfrom
codex/symphony-thread-comment-loop
Jun 1, 2026
Merged

[codex] Add Codex thread links and Linear comment resumes#84
yansenzhou-oai merged 1 commit into
mainfrom
codex/symphony-thread-comment-loop

Conversation

@yansenzhou-oai

Copy link
Copy Markdown
Contributor

Context

Need Symphony to keep a first-class Codex thread loop for Linear issues, including human follow-up comments and terminal cleanup.

TL;DR

Adds Codex thread links, thread archiving, and Linear comment resume support.

Summary

  • Add Codex app-server proxy support with thread owner metadata and archive calls.
  • Track thread IDs, comment cursors, and blocked human-input state in the orchestrator.
  • Fetch Linear comments and resume blocked or continued runs with human follow-up prompts.
  • Document the workflow/spec updates and cover stale empty workspace repair.

Alternatives

  • Keep comments entirely inside Codex workpads, but Linear replies stay invisible to Symphony.
  • Leave thread handling as plain dashboard text, but codex:// links make app handoff direct.

Test Plan

  • make -C elixir all
  • git diff --check
  • rg -n "lin_api_|ZtX3ZP" .

@yansenzhou-oai
yansenzhou-oai merged commit 68a18a7 into main Jun 1, 2026
2 checks passed
@yansenzhou-oai
yansenzhou-oai deleted the codex/symphony-thread-comment-loop branch June 1, 2026 02:42
yansenzhou-oai added a commit that referenced this pull request Jun 1, 2026
#### Context

Revert PR #84 after it was merged so Symphony main returns to the
previous workflow and implementation.

#### TL;DR

*Reverts the Codex thread link and Linear comment resume changes from PR
#84.*

#### Summary

- Revert the squash merge commit from PR #84.
- Remove the added Linear comment resume and Codex thread
ownership/archive behavior.
- Restore the previous docs, workflow config, app-server client, and
tests.

#### Alternatives

- Patch forward selectively, but the request was to revert the merged
change.

#### Test Plan

- [x] `git diff --check HEAD~1..HEAD`
- [x] `rg -n "lin_api_|ZtX3ZP" .`
- [x] `mix pr_body.check --file /tmp/symphony_revert_pr_body.md`
PouryaNoufallah96 added a commit to PouryaNoufallah96/symphony that referenced this pull request Jun 4, 2026
Sync with upstream (6 commits): README/docs + smoke-test notes, and the
networkAccess example-doc change (openai#65). PR openai#84 (Codex thread links + Linear
comment resumes) was added and then fully reverted upstream (openai#85), so this
merge carries no net code change — markdown only. Fork customizations
(team_key, Claude Code backend, ADO/GitHub adapters, Solid assigns) intact.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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