Restore the committed embed placeholder index

The built web/dist index.html was committed over the placeholder and referenced an uncommitted hashed asset. Keep this change limited to CI. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Drop CI build-info step; -trimpath omits ldflags from it
2026-10-03 00:34:16 -04:00 · 2026-10-03 00:30:52 -04:00 · 2026-10-03 00:30:32 -04:00
1 changed files with 9 additions and 37 deletions
@@ -1,6 +1,11 @@
 # Runs on every push and pull request (no branch filter) so feature branches,
 # their pull requests, and dev/main are all checked. Runs directly on the
 # self-hosted act_runner; no container: key.
+#
+# The Makefile is the single build definition: CI runs `make test` and
+# `make build` so it fails whenever either target would, and the binary
+# carries the Makefile's -trimpath and version-stamped -ldflags (the
+# `make build` log shows the -X main.version=... value).
 name: CI

 on: [push, pull_request]
@@ -16,45 +21,12 @@ jobs:
          go-version-file: go.mod
          cache: false

-      - name: gofmt
-        run: |
-          unformatted="$(gofmt -l .)"
-          if [ -n "$unformatted" ]; then
-            echo "gofmt needed:"
-            echo "$unformatted"
-            exit 1
-          fi
-
-      - name: go vet
-        run: go vet ./...
-
-      - name: go test
-        run: go test -race ./...
-
      - uses: actions/setup-node@v4
        with:
          node-version: "22"

-      - name: npm ci
-        working-directory: web
-        run: npm ci
+      - name: make test
+        run: make test

-      - name: npm run typecheck
-        working-directory: web
-        run: npm run typecheck
-
-      - name: npm test
-        working-directory: web
-        run: npm test
-
-      - name: npm run build
-        working-directory: web
-        run: npm run build
-
-      # Embed the real PWA, as `make embed` does, then build the static binary.
-      - name: go build (embedded PWA)
-        run: |
-          rm -rf internal/frontend/dist
-          mkdir -p internal/frontend/dist
-          cp -R web/dist/. internal/frontend/dist/
-          CGO_ENABLED=0 go build -o bin/mealquest ./cmd/server
+      - name: make build
+        run: make build
Author	SHA1	Message	Date
dev_agentandClaude Opus 5.5	b10257625f	Restore the committed embed placeholder index CI / ci (push) Successful in 25s Details CI / ci (pull_request) Successful in 26s Details Independent review Approved — 0 high findings (#9) Details The built web/dist index.html was committed over the placeholder and referenced an uncommitted hashed asset. Keep this change limited to CI. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>	2026-10-03 00:34:16 -04:00
dev_agentandClaude Opus 5.5	49743f4c83	Drop CI build-info step; -trimpath omits ldflags from it CI / ci (push) Successful in 25s Details CI / ci (pull_request) Successful in 26s Details Independent review Not signed off — 0 high, 1 medium (#9) Details Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>	2026-10-03 00:30:52 -04:00
dev_agentandClaude Opus 5.5	d1ab3932f8	Run make test and make build in CI CI duplicated the Makefile's steps and built the binary without -trimpath or the version-stamped -ldflags, so it could pass while make build failed and the CI binary reported dev. Make the Makefile the single build definition: CI now runs make test and make build, then prints the binary's build info as evidence of the version stamp. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>	2026-10-03 00:30:32 -04:00