Repository navigation
feat(harness): add OpenCrabs provider (ACP over stdio) #343
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Draft
moneyacademyKE
wants to merge
18
commits into
hardbeat920:main
Choose a base branch
from
moneyacademyKE:opencrabs-adapter-v0.1.52
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from all commits
Commits
Show all changes
18 commits
Select commit
Hold shift + click to select a range
5679192
feat(opencrabs): port OpenCrabs ACP harness adapter to v0.1.52 archit…
moneyacademyKE edd5e69
fix(opencrabs): resolve CodeRabbit review findings
moneyacademyKE 6d8e273
fix(opencrabs): close five interaction gaps from the round-3 audit
moneyacademyKE 9409e6e
fix(opencrabs): make the resume window honest
moneyacademyKE 400705d
fix(opencrabs): translate the ACP usage size field into the meter's w…
moneyacademyKE 2cc0fef
fix(opencrabs): address review — defer resume notice until fallback s…
moneyacademyKE bdddfff
fix(opencrabs): omit --model for the Default catalog placeholder
moneyacademyKE 2c3b9c4
feat(opencrabs): align adapter with v0.3.0 configurable-binary archit…
moneyacademyKE c0aaed8
fix(opencrabs): name connect failures, surface mode-switch errors; sp…
moneyacademyKE d9ad2ac
fix(models): label the OpenCrabs pre-connect entry Configured model
moneyacademyKE 23f70f5
fix(assets): redraw the OpenCrabs crab for 16px legibility
moneyacademyKE 6951c7a
ci(opencrabs): build windows and linux packages on the fork without u…
moneyacademyKE a8ce328
ci(opencrabs): trigger cross builds from fork release tags
moneyacademyKE 55e939a
opencrabs: subagent cards, nested child steps, inline images
moneyacademyKE e815cba
feat(opencrabs): render cross-surface mirror pushes in resumed sessions
moneyacademyKE f003594
fix(opencrabs): read image metadata off the wire, not node:fs
moneyacademyKE c99433e
ci(release): build rpm packages alongside deb and AppImage
moneyacademyKE 8ce3e2a
ci(release): build the rpm inside an EL 10 container, not on Ubuntu
moneyacademyKE File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,138 @@ | ||
| name: Cross release (fork) | ||
|
|
||
| on: | ||
| workflow_dispatch: | ||
| push: | ||
| tags: | ||
| - "v*-opencrabs.*" | ||
|
|
||
| permissions: | ||
| contents: read | ||
|
|
||
| jobs: | ||
| linux: | ||
| name: Linux packages | ||
| runs-on: ubuntu-22.04 | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
|
|
||
| - uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: 20 | ||
| cache: npm | ||
|
|
||
| - uses: dtolnay/rust-toolchain@stable | ||
|
|
||
| - name: Install Linux system dependencies | ||
| run: bash scripts/install-linux-deps-debian.sh | ||
|
|
||
| - run: npm ci | ||
|
|
||
| - name: Build Linux packages | ||
| run: | | ||
| set -euo pipefail | ||
| printf '%s\n' '{"bundle":{"createUpdaterArtifacts":false}}' > "$RUNNER_TEMP/tauri.fork.conf.json" | ||
| npm run build:linux -- --config "$RUNNER_TEMP/tauri.fork.conf.json" | ||
|
|
||
| - name: Stage Linux packages | ||
| run: | | ||
| set -euo pipefail | ||
| shopt -s nullglob | ||
| mkdir -p release-artifacts | ||
| cp target/release/bundle/deb/*.deb release-artifacts/ | ||
| cp target/release/bundle/appimage/*.AppImage release-artifacts/ | ||
|
|
||
| - name: Upload Linux packages | ||
| uses: actions/upload-artifact@v4 | ||
| with: | ||
| name: linux-packages | ||
| path: release-artifacts/ | ||
| if-no-files-found: error | ||
|
|
||
| linux-rpm: | ||
| name: Linux rpm package | ||
| runs-on: ubuntu-latest | ||
| # Same recipe as upstream: build inside EL 10 so the rpm's glibc | ||
| # requirement matches the oldest advertised target; an Ubuntu-linked | ||
| # rpm would install on EL 10 and then fail to load. | ||
| container: almalinux:10 | ||
| steps: | ||
| - name: Install git | ||
| run: dnf install -y git | ||
|
|
||
| - uses: actions/checkout@v4 | ||
|
|
||
| - uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: 20 | ||
| cache: npm | ||
|
|
||
| - uses: dtolnay/rust-toolchain@stable | ||
|
|
||
| - name: Install Enterprise Linux system dependencies | ||
| run: bash scripts/install-linux-deps-fedora.sh | ||
|
|
||
| - run: npm ci | ||
|
|
||
| - name: Build rpm package | ||
| run: npm run build:fedora | ||
|
|
||
| - name: Verify and stage the rpm | ||
| run: | | ||
| set -euo pipefail | ||
| shopt -s nullglob | ||
| bundle=(target/release/bundle/rpm/*.rpm) | ||
| if (( ${#bundle[@]} != 1 )); then | ||
| echo "Expected exactly one .rpm" >&2 | ||
| exit 1 | ||
| fi | ||
| requires="$(rpm -qp --requires "${bundle[0]}")" | ||
| for dep in webkit2gtk4.1 gtk3 libappindicator-gtk3 librsvg2 openssl-libs; do | ||
| if ! grep -qx "$dep" <<<"$requires"; then | ||
| echo "rpm is missing Requires: $dep" >&2 | ||
| exit 1 | ||
| fi | ||
| done | ||
| mkdir -p release-artifacts | ||
| cp "${bundle[0]}" release-artifacts/ | ||
|
|
||
| - name: Upload rpm package | ||
| uses: actions/upload-artifact@v4 | ||
| with: | ||
| name: linux-rpm-package | ||
| path: release-artifacts/ | ||
| if-no-files-found: error | ||
|
|
||
| windows: | ||
| name: Windows packages | ||
| runs-on: windows-latest | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
|
|
||
| - uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: 20 | ||
| cache: npm | ||
|
|
||
| - uses: dtolnay/rust-toolchain@stable | ||
|
|
||
| - run: npm ci | ||
|
|
||
| - name: Build Windows installer | ||
| shell: pwsh | ||
| run: | | ||
| '{"bundle":{"createUpdaterArtifacts":false}}' | Set-Content -Encoding utf8 "$env:RUNNER_TEMP\tauri.fork.conf.json" | ||
| npx tauri build --bundles nsis --config "$env:RUNNER_TEMP\tauri.fork.conf.json" | ||
|
|
||
| - name: Stage Windows packages | ||
| shell: pwsh | ||
| run: | | ||
| New-Item -ItemType Directory -Force "release-artifacts" | Out-Null | ||
| Copy-Item "target/release/bundle/nsis/*.exe" "release-artifacts/" | ||
|
|
||
| - name: Upload Windows packages | ||
| uses: actions/upload-artifact@v4 | ||
| with: | ||
| name: windows-packages | ||
| path: release-artifacts/ | ||
| if-no-files-found: error |
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
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
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
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
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
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
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
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
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
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
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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.