Skip to content

Add fractal-chatgpt binary and keep st-chatgpt alias - #4636

Merged
github-merge-queue[bot] merged 5 commits into
mainfrom
schickling-assistant/2026-11-01-fractal-chatgpt-reader
Nov 4, 2026
Merged

github-merge-queue[bot] merged 5 commits into
mainfrom
schickling-assistant/2026-11-01-fractal-chatgpt-reader

Conversation

@schickling-assistant

@schickling-assistant schickling-assistant commented Nov 1, 2026 •

Copy link
Copy Markdown
Contributor

Problem

Source builds had only the legacy st-chatgpt executable name.

Goal

Build fractal-chatgpt from the same adapter source while keeping st-chatgpt buildable for the current Nix package.

Decisions

Add a second Cargo target, st-chatgpt, with the same src/main.rs. The comment marks it as the bridge until the package-name cutover. Nix package attributes, scripts, and the prototype build command stay unchanged here.

Docs describe binary fractal-chatgpt (st-chatgpt kept as an alias binary) and still point Nix consumers at #st-chatgpt.

The package attribute, symlink, and exercise path move to the stacked PR.

Verification

cargo build -p fractal-chatgpt and cargo test -p fractal-chatgpt are running for this head on the admitted build lane. Results follow when available.

Complexity

No new abstraction. One additional binary target.

Concerns

None beyond the temporary second target.

Friction & bottlenecks

None.

Follow-ups

Package cutover is the stacked PR. Remove the alias target at contraction (see #3032).

References

See #3032.

Posted on behalf of @schickling
field value
agent_identity agent/compoundingtech/st-chatgpt
session 01a1787a
agent_persona generalist
agent_supervisor unavailable
agent_tool OMP
agent_tool_version 18.10.2
agent_runtime OMP 18.10.2
agent_model anthropic/claude-opus-5-5
worktree smalltalk/main

@schickling-assistant schickling-assistant changed the title Rename ChatGPT adapter and accept both paired-client sockets Rename ChatGPT adapter binary/package to fractal-chatgpt (st-chatgpt bridge) Nov 2, 2026
@schickling-assistant

Copy link
Copy Markdown
Contributor Author

Acceptance evidence at b65d0c6669b8f1e72357c3d3c96ff3343321c0af on dev3, admitted by gate-slot build lanes:

  • nix develop --command cargo test -p fractal-chatgpt — exit 0; 21 lib tests passed, 0 failed.
  • nix develop --command cargo clippy -p fractal-chatgpt --all-targets -- -D warnings — exit 0.
  • nix build .#fractal-chatgpt — exit 0.
  • nix build .#st-chatgpt — exit 0.
  • Both outputs contain bin/fractal-chatgpt and bin/st-chatgpt -> fractal-chatgpt.
  • result-st-chatgpt/bin/st-chatgpt --help — exit 0.

Socket-name acceptance remains owned by #4639 and is unchanged here.

Posted on behalf of @schickling
field value
agent_identity agent/compoundingtech/st-chatgpt
session 01a1787a
agent_persona generalist
agent_supervisor unavailable
agent_tool OMP
agent_tool_version 18.10.2
agent_runtime OMP 18.10.2
agent_model anthropic/claude-opus-5-5
worktree smalltalk/main

@schickling-assistant

Copy link
Copy Markdown
Contributor Author

Independent review (exact head 2e896cb): GO, no open findings. Read-only reviewer run by st-chatgpt (agent/compoundingtech/st-chatgpt) as a subagent.

  • The prior NO-GO at b65d0c6 is resolved. docs/st3/client-v0/README.md and scripts/chatgpt-prototype-e2e.py now use the fractal-chatgpt binary. No other source-build consumer of the old binary name remains.
  • The Nix bridge (packages.st-chatgpt, bin/st-chatgpt -> fractal-chatgpt) follows the existing live-migration pattern. The release note matches the schema.
  • The docs hunks are disjoint from Cut default socket readers and docs over to Fractal names #4639 (socket allowlist). Both PRs are predicted to merge cleanly in either order.

Author evidence: cargo test -p fractal-chatgpt passed 21 tests; clippy -D warnings rc 0; nix build .#fractal-chatgpt and .#st-chatgpt rc 0; st-chatgpt --help rc 0.

Posted on behalf of @schickling
field value
agent_identity agent/compoundingtech/st-chatgpt
session 01a1787a
agent_persona generalist
agent_supervisor unavailable
agent_tool OMP
agent_tool_version 18.10.2
agent_runtime OMP 18.10.2
agent_model anthropic/claude-opus-5-5
worktree smalltalk/main

…2026-11-01-fractal-chatgpt-reader

# Conflicts:
#	docs/st3/client-v0/README.md
@schickling-assistant

Copy link
Copy Markdown
Contributor Author

Independent review (exact head a40a58e): GO, no findings. Delta review by a read-only reviewer that st-chatgpt (agent/compoundingtech/st-chatgpt) ran as a subagent.

  • The head is a two-parent merge of the reviewed head 2e896cb and main 63052f4.
  • origin/main...a40a58e7bdbafc637a8bb000120788e85f455423 keeps exactly the reviewed 6-file scope. In the README conflict resolution, main's -p fractal-cli stays and only --bin st-chatgpt changes to --bin fractal-chatgpt.
  • Main adds no new consumer of the old binary name.

Author proof at this head (dev3 gate-slot build lane): cargo build -p fractal-chatgpt --bin fractal-chatgpt rc 0; cargo test -p fractal-chatgpt passed 21 of 21.

Posted on behalf of @schickling
field value
agent_identity agent/compoundingtech/st-chatgpt
session 01a1787a
agent_persona generalist
agent_supervisor unavailable
agent_tool OMP
agent_tool_version 18.10.2
agent_runtime OMP 18.10.2
agent_model anthropic/claude-opus-5-5
worktree smalltalk/main

@schickling-assistant
schickling-assistant marked this pull request as ready for review November 2, 2026 06:43
@schickling-assistant schickling-assistant changed the title Rename ChatGPT adapter binary/package to fractal-chatgpt (st-chatgpt bridge) Add fractal-chatgpt binary and keep st-chatgpt alias Nov 2, 2026
@schickling-assistant

Copy link
Copy Markdown
Contributor Author

Independent review (exact head 9f76b75): GO, no findings. Read-only reviewer run by st-chatgpt (agent/compoundingtech/st-chatgpt) as a subagent.

Author proof at this head: cargo build -p fractal-chatgpt builds both binaries; cargo test -p fractal-chatgpt passed 21 of 21.

Posted on behalf of @schickling
field value
agent_identity agent/compoundingtech/st-chatgpt
session 01a1787a
agent_persona generalist
agent_supervisor unavailable
agent_tool OMP
agent_tool_version 18.10.2
agent_runtime OMP 18.10.2
agent_model anthropic/claude-opus-5-5
worktree smalltalk/main

@myobie

myobie commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

Back with the author at head 9f76b750edb0: required checks genie-freshness and linux-gate are red because the evidence gate has no signature for this tree: genie-freshness: no signature for tree 9c9302be03ca. Our pull requests do not run CI: sign it with scripts/st-test --all --push origin. Since #4358 (signed evidence for assistant pull requests) merged at 18:49Z, a head from this lane needs signed evidence for its tree before the required checks pass. Next step: run scripts/st-test --all --push origin on the current head so its signed notes are pushed with the branch (a new head needs this again). Failed run: https://github.com/compoundingtech/fractal/actions/runs/41402327933

@schickling-assistant

Copy link
Copy Markdown
Contributor Author

@myobie, st-chatgpt (agent/compoundingtech/st-chatgpt) answers your comment 6571150769 at head 9f76b750edb0c96af7afa97317489006a6945887.

The red genie-freshness and linux-gate checks are the expected missing-signature state. Under the signing rule from the Compounding Manager since #4358, author agents do not run scripts/st-test --all --push. Mergemaster (agent/compoundingtech/mergemaster) is the only signer, once per head and after an independent GO, until dotfiles#8141 lands.

  • Signing request for this exact head: sent to Mergemaster (message fea7285b), with the GO in comment 6571123070.
  • This head is already split for signing: it touches no flake.nix, scripts/, .st/, .github/ or genie inputs. The flake.nix part moved to Publish fractal-chatgpt Nix package with st-chatgpt bridge #4688, which is a person-route case.
  • st-chatgpt will not move the head until Mergemaster has signed it.
Posted on behalf of @schickling
field value
agent_identity agent/compoundingtech/st-chatgpt
session 01a1787a
agent_persona generalist
agent_supervisor unavailable
agent_tool OMP
agent_tool_version 18.10.2
agent_runtime OMP 18.10.2
agent_model anthropic/claude-opus-5-5
worktree smalltalk/main

@github-merge-queue
github-merge-queue Bot merged commit 2e0b593 into main Nov 4, 2026
66 of 68 checks passed
@schickling-assistant

Copy link
Copy Markdown
Contributor Author

Parked: landed via hosted copy #4982 (merge commit 2e0b593). Same reviewed content. Kept open for Nathan's post-merge review (#4898). Compounding Manager.

Posted on behalf of @schickling
field value
agent_identity agent/compoundingtech/compounding-manager
session 01a1d992
agent_persona generalist
agent_supervisor unavailable
agent_tool OMP
agent_tool_version 18.12.5
agent_runtime OMP 18.12.5
agent_model anthropic/claude-opus-5-5
worktree unknown

This branch was previously deployed

1 inactive deployment
evidence-signing — 9f76b750 Deployed Nov 4, 2026 by schickling via evidence-note-linux-gate #15727
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.

3 participants