Skip to content

fix: last stale platform/ refs in scripts, tests, compose #943

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.

Sign up for GitHub

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

Merged
HongmingWang-Rabbit merged 1 commit into from
Apr 18, 2026
Merged

fix: last stale platform/ refs in scripts, tests, compose #943

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions docker-compose.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -103,12 +103,12 @@ services:
platform:
build:
# Build context MUST be repo root, not ./platform — the Dockerfile
# COPYs `platform/migrations`, `platform/go.mod`,
# COPYs `workspace-server/migrations`, `workspace-server/go.mod`,
# `workspace-configs-templates/` etc. via repo-relative paths so it
# can bake in templates + migrations alongside the platform binary.
# When context was ./platform earlier, docker silently cached an
# earlier image (the COPY platform/migrations resolved to nothing
# under ./platform/, so layers stopped invalidating) — manifested
# earlier image (the COPY workspace-server/migrations resolved to nothing
# under ./workspace-server/, so layers stopped invalidating) — manifested
# as migration 023 not landing after PR #417 merged. CI workflow
# already uses context=. , this aligns local with CI.
context: .
Expand Down
2 changes: 1 addition & 1 deletion infra/scripts/setup.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ KEA=$(docker compose -f "$ROOT_DIR/docker-compose.infra.yml" exec -T redis redis
echo " notify-keyspace-events = $KEA"

echo "==> Running migrations..."
MIGRATIONS_DIR="$ROOT_DIR/platform/migrations"
MIGRATIONS_DIR="$ROOT_DIR/workspace-server/migrations"
if [ -d "$MIGRATIONS_DIR" ]; then
for f in "$MIGRATIONS_DIR"/*.sql; do
echo " Applying $(basename "$f")..."
Expand Down
2 changes: 1 addition & 1 deletion tests/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ This repo uses the standard monorepo testing convention: **unit tests live with

| Scope | Location |
|---|---|
| Go unit + integration (platform, CLI, handlers) | `platform/**/*_test.go` — run with `cd platform && go test -race ./...` |
| Go unit + integration (platform, CLI, handlers) | `workspace-server/**/*_test.go` — run with `cd workspace-server && go test -race ./...` |
| TypeScript unit (canvas components, hooks, store) | `canvas/src/**/__tests__/` — run with `cd canvas && npm test -- --run` |
| TypeScript unit (MCP server handlers) | `mcp-server/src/__tests__/` — run with `cd mcp-server && npx jest` |
| Python unit (workspace runtime, adapters) | `workspace-template/tests/` — run with `cd workspace-template && python3 -m pytest` |
Expand Down