Repository navigation
Findings of the first bridge built from the template: make test no longer reaps every coraine, a copy no longer pushes to the template, a refused Channel says why - #1
Merged
Conversation
… make test no longer reaps every coraine on the host, a copy no longer pushes to the template, the TODO grep finds the README and the CI, and a refused Channel says why in the plugin's words - make test: COR_TEST_NO_REAP=on - corTest's start-up reaping (killall -9 coraine) is for coraine's own suite; the tests stop their broker by pid - starting a bridge: an export (git archive) and git init, not cp -r - which carried the template's .git, its history and an origin pushing to corBridgeTemplate; rename.sh warns when origin is the template - grep -rn --exclude-dir=.git TODO . - the README (marked: its intro, "Where each piece goes" and "Configuration" describe the example transport) and ci.yml carry TODOs too - channelStatusIn (bridge ABI 12, under #if: builds against older headers): the transport's own reason on a refused Channel - bridge_tmpl's '../escape' shows it; the README says how and when - README: DataChannels (a transport whose messages are rows; coraine COR_FEATURE_DATA_CHANNELS), publish NULL for a bridge that only reads, the catch-all entity on unless "defaultEntity": false Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…hout the corJson tool, built but not on PATH here (as coraine's and corCkanBridge's ci.yml set it) Co-Authored-By: Claude Opus 5.5 <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
1 participant
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.
Merge after: nothing - needs coraine with
channelStatusIn(SEAMWARE/coraine#360, merged) and corBridge ABI 12 (corBridge#13, merged)What building the first bridge from the template (corCkanBridge) found:
make testno longer kills every coraine on the host. corTest's start-up reaping (killall -9 coraine) is meant for coraine's own suite. Here it's off (COR_TEST_NO_REAP=on), and the tests stop their own broker by pid.git archive) plusgit init, notcp -r. The copy carried the template's.git, its history, and anoriginthat pushed to corBridgeTemplate.rename.shwarns whenoriginis the template.grep -rn --exclude-dir=.git TODO .) now finds the README andci.ymltoo.channelStatusIn, under#iffor headers older than ABI 12, andbridge_tmpl's../escapecase shows it.COR_FEATURE_DATA_CHANNELS): a transport whose messages are records; the record's time is 0 when it has none;publishNULL for a bridge that only reads;"defaultEntity": false.Tested: rebased on main and built against the main cor libs.
make contractis OK, andmake testgives 1/1, twice in a row, against coraine main (release build with #360).This branch was pushed on 2026-10-10 and never had a PR. It waited for
channelStatusInand the record Channels, which landed as #360 and #361 (DataChannels).🤖 Generated with Claude Code