qaify-playwright-reporter 0.4.1 → 0.4.2

package/dist/audit.d.ts

@@ -1,4 +1,4 @@
-import type { IngestClient } from './client.js';
+import { type IngestClient } from './client.js';
 export declare const AREAS: readonly ["ui", "business_logic", "api_contract", "validation", "error_handling", "security", "performance"];
 export declare const SEVERITIES: readonly ["critical", "high", "medium", "low"];
 export declare const ROLES: readonly ["fe", "be"];
@@ -295,5 +295,7 @@ export declare function auditContext(client: IngestClient, auditId: string, dir:
  * Screenshot (temuan + galeri halaman) hanya diambil dari dalam folder evidence (path di luar folder itu diabaikan).
  */
 export declare function auditSubmit(client: IngestClient, auditId: string, claudeOutput: string, evidenceDir: string, commits: Record<string, string>, log?: (msg: string) => void): Promise<any>;
+/** Penyebab screenshot tidak terunggah, agar laporan "Tanpa screenshot" tidak terjadi diam-diam. */
+export declare function uploadProblems(res: any, unsigned: number, rejected: number, putError: string | null): string[];
 /** Commit per role dari context.json (dibuat audit-context). */
 export declare function readCommits(contextFile: string): Promise<Record<string, string>>;

package/dist/audit.js

@@ -6,6 +6,7 @@ import { execFile } from 'node:child_process';
 import { mkdir, readFile, stat, writeFile } from 'node:fs/promises';
 import { extname, join, resolve, sep } from 'node:path';
 import { promisify } from 'node:util';
+import { warn } from './client.js';
 export const AREAS = ['ui', 'business_logic', 'api_contract', 'validation', 'error_handling', 'security', 'performance'];
 export const SEVERITIES = ['critical', 'high', 'medium', 'low'];
 export const ROLES = ['fe', 'be'];
@@ -304,15 +305,40 @@ async function uploadScreenshots(client, base, refs, evidenceDir, log) {
         const entries = [...files.entries()];
         const res = await client.post(`${base}/artifacts`, { items: entries.map(([, f]) => ({ mime: f.mime, size_bytes: f.size })) });
         const slots = Array.isArray(res?.uploads) ? res.uploads : [];
+        let unsigned = 0;
+        let rejected = 0;
         for (const [i, [ref, file]] of entries.entries()) {
             const slot = slots[i];
-            if (slot?.url && (await client.put(slot.url, slot.headers ?? {}, await readFile(file.path))))
+            if (!slot?.url)
+                unsigned++;
+            else if (await client.put(slot.url, slot.headers ?? {}, await readFile(file.path)))
                 uploaded.set(ref, slot.path);
+            else
+                rejected++;
         }
         log(`qa-factory: ${uploaded.size}/${files.size} screenshot terunggah`);
+        for (const problem of uploadProblems(res, unsigned, rejected, client.uploadError()))
+            warn(problem);
     }
     return uploaded;
 }
+/** Penyebab screenshot tidak terunggah, agar laporan "Tanpa screenshot" tidak terjadi diam-diam. */
+export function uploadProblems(res, unsigned, rejected, putError) {
+    const problems = [];
+    if (unsigned) {
+        problems.push(!res
+            ? `${unsigned} screenshot tidak diunggah: API Qaify menolak atau tidak menjawab permintaan URL upload.`
+            : res.reason === 'storage_not_configured'
+                ? `${unsigned} screenshot tidak diunggah: API Qaify tidak bisa menandatangani upload. Isi ARTIFACT_DISK dan AWS_ACCESS_KEY_ID / AWS_SECRET_ACCESS_KEY / AWS_BUCKET / AWS_ENDPOINT (R2) di env API, lalu deploy ulang.`
+                : res.reason === 'audit_inactive'
+                    ? `${unsigned} screenshot tidak diunggah: audit sudah tidak aktif (dibatalkan atau melewati batas waktu) saat hasil dikirim.`
+                    : `${unsigned} screenshot tidak mendapat URL upload (format bukan PNG/JPEG atau melebihi batas ukuran).`);
+    }
+    if (rejected) {
+        problems.push(`${rejected} screenshot ditolak storage (${putError ?? 'gagal'}). Cek izin token R2 (Object Read & Write) serta AWS_ENDPOINT / AWS_BUCKET di env API.`);
+    }
+    return problems;
+}
 /** Commit per role dari context.json (dibuat audit-context). */
 export async function readCommits(contextFile) {
     try {

package/dist/client.d.ts

@@ -31,6 +31,8 @@ export interface IngestClient {
     post(path: string, body: unknown): Promise<any>;
     get(path: string): Promise<any>;
     put(url: string, headers: Record<string, string>, body: Uint8Array): Promise<boolean>;
+    /** Penyebab PUT terakhir yang gagal (mis. "HTTP 403 AccessDenied"), null bila belum pernah gagal. */
+    uploadError(): string | null;
 }
 export declare const sleep: (ms: number) => Promise<void>;
 export declare const SETTINGS_HINT = "Cek Settings \u2192 Secrets and variables \u2192 Actions: QA_FACTORY_TOKEN di tab Secrets, QA_FACTORY_URL di tab Variables.";

package/dist/client.js

@@ -56,6 +56,7 @@ export function createClient(config = {}) {
     let problem = null;
     let httpStatus = null;
     let notFound = 0;
+    let uploadError = null;
     async function send(method, path, body) {
         if (!url || !token)
             return null;
@@ -109,11 +110,18 @@ export function createClient(config = {}) {
         async put(target, headers, body) {
             try {
                 const res = await doFetch(target, { method: 'PUT', headers, body, signal: AbortSignal.timeout(UPLOAD_TIMEOUT_MS) });
+                if (!res.ok) {
+                    // S3/R2 menjelaskan penolakan di <Code> (AccessDenied, SignatureDoesNotMatch, NoSuchBucket, …)
+                    const code = /<Code>([^<]+)<\/Code>/.exec(await res.text().catch(() => ''))?.[1];
+                    uploadError = `HTTP ${res.status}${code ? ` ${code}` : ''}`;
+                }
                 return res.ok;
             }
-            catch {
+            catch (e) {
+                uploadError = String(e?.cause ?? e?.message ?? e);
                 return false;
             }
         },
+        uploadError: () => uploadError,
     };
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "qaify-playwright-reporter",
-  "version": "0.4.1",
+  "version": "0.4.2",
   "description": "Reporter Playwright untuk QA Factory (Qaify): kirim hasil test dan artefak ke /api/ingest, tanpa pernah menggagalkan test.",
   "license": "MIT",
   "author": "ahmdkenz",