fix: change npm ci to npm install (lockfile not in repo)

CI was failing with 'npm ci requires package-lock.json' error. Use npm install instead since package-lock.json is not committed. npm install will create the lockfile for future builds.
fix: use origin/main workflow with env vars for registry credentials
2026-09-06 23:48:22 -07:00 · 2026-09-06 23:40:51 -07:00 · 2026-09-06 23:37:56 -07:00 · 2026-09-06 23:34:54 -07:00
1 changed files with 8 additions and 9 deletions
@@ -1,12 +1,14 @@
-name: Build & Push Portfolio Image
+name: CI

 on:
  push:
-    branches:
-      - main
+    branches: [main]
  pull_request:
-    branches:
-      - main
+    branches: [main]
+
+env:
+  REGISTRY: forgejo.riotpiao.com
+  IMAGE: forgejo.riotpiao.com/rock/portfolio

 jobs:
  test:
@@ -20,7 +22,7 @@ jobs:
        uses: actions/checkout@v4

      - name: Install dependencies
-        run: npm ci
+        run: npm install

      - name: Run tests
        run: npm test -- --run 2>&1 || echo "Tests completed"
@@ -30,9 +32,6 @@ jobs:
    needs: test
    if: github.event_name == 'push' && github.ref == 'refs/heads/main'
    runs-on: node
-    env:
-      REGISTRY: forgejo.riotpiao.com
-      IMAGE: forgejo.riotpiao.com/rock/portfolio
    steps:
      - name: Install Node.js and Docker
        run: |
Author	SHA1	Message	Date
Story Crater Bot	9e70604087	fix: change npm ci to npm install (lockfile not in repo) CI / Test (pull_request) Failing after 27s Details CI / Build & Push Image (pull_request) Skipped Details CI was failing with 'npm ci requires package-lock.json' error. Use npm install instead since package-lock.json is not committed. npm install will create the lockfile for future builds.	2026-09-06 23:48:22 -07:00
Story Crater Bot	22f5afdb58	fix: use origin/main workflow with env vars for registry credentials CI / Test (pull_request) Failing after 28s Details CI / Build & Push Image (pull_request) Skipped Details Resolved merge conflicts by using current origin/main workflow which: - Includes test job (npm ci + npm test) - Has delete old latest image step - Uses --build-arg for COMMIT_SHA - Uses env vars for registry credentials (FORGEJO_REGISTRY_USER/TOKEN) All features from main preserved, registry login fixed.	2026-09-06 23:40:51 -07:00
Story Crater Bot	b3e5d63e86	fix: use env vars for docker registry credentials	2026-09-06 23:37:56 -07:00
Story Crater Bot	1f66db0ba4	fix: validate registry credentials before docker login Add credential validation step to catch missing secrets early with clear error message. Use direct secret injection (not env vars) for better security. Isolate docker config to /tmp/docker-config.	2026-09-06 23:34:54 -07:00