From 9af49921c3dfdd95569b9751679f9447a84cb6fb Mon Sep 17 00:00:00 2001 From: Renaud Laborbe <150821561+BluegReeno@users.noreply.github.com> Date: Sat, 25 Jul 2026 00:02:18 +0200 Subject: [PATCH] Fix: document root Makefile targets in README build/deploy section (#59) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit README's "Develop & deploy" section showed `make deploy` once inline with no description and never surfaced `make test` / `make verify` or the runbook location. Changes: - List all three root Makefile targets (deploy/verify/test) with descriptions mirrored verbatim from the Makefile's `## ` comments - Point to CLAUDE.md § Migrations & deploy and docs/operations.md for the runbook - Drop the now-redundant inline `make deploy` line from the bash block Fixes #59 Co-Authored-By: Claude Opus 4.8 --- README.md | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/README.md b/README.md index 563f43f..1a9a8b9 100644 --- a/README.md +++ b/README.md @@ -274,10 +274,15 @@ hal/ ## Develop & deploy -```bash -# Deploy the MCP server (the actual product) — see docs/operations.md §2 for the full runbook -make deploy # supabase functions deploy hal-mcp --no-verify-jwt + verify_deploy.sh +Canonical entry points (root `Makefile`): + +- `make deploy` — deploy hal-mcp + verify it answers +- `make verify` — probe the live hal-mcp function (no redeploy) +- `make test` — offline checks — deno type-check (both fns) + offline pytest (mirrors CI) +Full runbook: [`CLAUDE.md`](./CLAUDE.md) § Migrations & deploy, [`docs/operations.md`](./docs/operations.md). + +```bash # Apply a halcrm_* migration — use the Supabase MCP apply_migration tool or the # dashboard SQL editor. The Supabase CLI push is NOT safe on this shared # partial-ledger DB; see docs/operations.md §1 for the runbook.