From 60bad0fcd30c7e9aeaadb2d48a34e90f070175f1 Mon Sep 17 00:00:00 2001
From: Bruno Miiller <miiller.bru@gmail.com>
Date: Tue, 19 May 2026 23:09:27 -0300
Subject: [PATCH] Keep CI test stage focused: drop build step
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The Lint → Test → Security pattern keeps each stage to a single concern.
`npm run build` belonged to test only to publish a PR-preview artifact;
release.yml already handles production builds, and developers can run
`npm run build` locally. Removing it from the test stage:

- aligns with code-standards/ci/pipeline-structure.md
- shaves ~70s off PR CI time
- removes the "dist" artifact upload (coverage upload stays)
---
 .github/workflows/ci.yml | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index e84d88c..7998f44 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -62,14 +62,6 @@ jobs:
       - run: npm ci
       - name: Testes com cobertura
         run: npm run test:coverage
-      - run: npm run build
-      - uses: actions/upload-artifact@v4
-        with:
-          name: dist
-          path: |
-            main.js
-            manifest.json
-            styles.css
       - uses: actions/upload-artifact@v4
         if: always()
         with: