Remove legacy references - #28
Merged
1 commit merged intoApr 9, 2025
Merged
Conversation
- Remove 'ai-config' references from README - Simplify ai-config.js implementation without warning messages - Improve README clarity
joelteply
added a commit
that referenced
this pull request
Jun 19, 2026
…t layer (#1689) The old Node/TS web client is being reinvented on the new client SDK (screenshot-as-spec, own modular container), not repaired. Its build is prone to directory-reshuffle breakage from the headless-Rust work — e.g. carl-install-smoke has been RED across all of canary because a tsx build step resolves browser-index.ts to tools/ instead of src/, so its `./browser/generated` import ENOENTs (the real, git-tracked file is src/browser/generated.ts). That dead-layer break aborted the whole public `curl install.sh | bash` before the headless core ever came up. Decouple both the installer and the smoke gate from that layer: install.sh: - Host-side jtag CLI bundle build is now BEST-EFFORT / non-blocking. On failure it warns loudly and CONTINUES — the headless Rust core is the install deliverable, brought up by the container runtime regardless. (Previously a hard `fail` aborted install on the Node build break.) carl-install-smoke.sh: - New PRIMARY gate: wait for the continuum-core IPC socket (/root/.continuum/sockets/continuum-core.sock, the same signal as the compose healthcheck) via `docker compose exec`. "Smoke passed" now means "the headless core serves IPC", independent of the Node web build. - The old widget-server :9003 render + jtag chat-e2e assertions are now ADVISORY (default-skipped, loud), re-enablable with CARL_CHECK_WEB_CLIENT=1 once the new client container lands and they're rewritten against it. Roadmap: #28 (the dead-layer path break, intentionally NOT deep-fixed), #29 (new Node/TS client SDK + app as a modular container on the new SDK). Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR removes unnecessary legacy references:
Test plan
This makes the documentation more straightforward for this new project.
🤖 Generated with Claude Code