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
+      - name: make test
-        working-directory: web
+        run: make test
        run: npm ci
-      - name: npm run typecheck
+      - name: make build
-        working-directory: web
+        run: make build
        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
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