feat: add local dashboard development flow - #197
Open
userAugustos wants to merge 12 commits into
Open
userAugustos wants to merge 12 commits into
userAugustos wants to merge 12 commits into
Conversation
|
Warning Review limit reachedNext included review available in 47 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (11)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Builds the core, api and web :dev images, syncs the workspace venv, and opens a shell where `canyonos` is this checkout's CLI. The host hot-reload dashboard moves to `bun run dashboard:dev`. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Resolve packages/cli/canyonos/resources.py and its tests to main's side. #198 made Core accept `gpu: 0` (manifest schema `_integer(..., 0)`, the deploy command's `_load_config`, and both runtimes already treat a falsy gpu as "no GPU"), so this branch's `_omit_zero_gpu` workaround and its tests are obsolete and are dropped with the merge. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017CiY8b3FNgfXCPQLR4PtdV
The hint still named canyonos_core/Dockerfile with the repo root as context, which no longer exists; the image is built by bun run canyonos:dev. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017CiY8b3FNgfXCPQLR4PtdV
The one-liner relied on POSIX ${VAR:-default} expansion, which Bun's own shell (bun run --shell=bun, the default on Windows) passes through literally, so the API got APP_ENV='${APP_ENV:-development}' and refused to start. The script applies the same defaults with the same empty-means-unset rule, next to canyonos:dev and canyonos:smoke.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017CiY8b3FNgfXCPQLR4PtdV
SHELL= made Bun.spawn fail on an empty executable name. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017CiY8b3FNgfXCPQLR4PtdV
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017CiY8b3FNgfXCPQLR4PtdV
- dashboard:dev still reads .docker/.env when present (Compose loads the project directory's .env); it only does not require it. - Its defaults override packages/api/.env and packages/web/.env, not just yield to the shell. - The config lives at config/global_controller.yaml in a project without .car, e.g. examples/portfolio. - A non-default workflow redis_port needs CANYONOS_REDIS_PORT. - Link DEV.md and docs/CONTRIBUTING.md to each other instead of repeating setup. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017CiY8b3FNgfXCPQLR4PtdV
docs/ is built by MkDocs in strict mode (#217), where a relative link to a file outside docs/ fails the build. Same form as the other out-of-tree links in this file. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017CiY8b3FNgfXCPQLR4PtdV
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
3 participants
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
bun run canyonos:dev: builds the Core, API and web images from this checkout (canyonos-*:dev), createspackages/cli/.envif missing, runsuv sync, and opens a shell wherecanyonosis the workspace CLI using those imagesbun run dashboard:dev(scripts/dashboard-dev.ts): local Postgres/Mailpit through Compose, then the API and Vite in watch mode with local defaults (fixed-code auth, no deploy worker, workflow Redis on127.0.0.1:6379); shell variables override themDEV.md, linked fromdocs/CONTRIBUTING.mdcanyonos -vprints the environment name (development/test) outside production; production output is unchangedThe earlier zero-GPU omission in the resource picker is gone: #198 made Core accept
gpu: 0, so the picker's default is valid as written.Test
bun run checkbun run test(CLI: 282 passed)uv run pytest packages/cli/tests(282 passed)canyonos deploy --serve falseagainst the LLM-free Epigenomics workflow; request completed and the host dev API stored 15 spans across all six agent operationsReview Focus
canyonos:dev/dashboard:devenvironment defaults and caller overrides