Skip to content

fix(ci): repair invalid refresh-vuln-db workflow (duplicate env key) - #265

Merged
Wolfvin merged 1 commit into
mainfrom
fix/refresh-vuln-db-workflow-invalid
Jul 13, 2026
Merged

fix(ci): repair invalid refresh-vuln-db workflow (duplicate env key)#265
Wolfvin merged 1 commit into
mainfrom
fix/refresh-vuln-db-workflow-invalid

Conversation

@Wolfvin

@Wolfvin Wolfvin commented Jul 13, 2026

Copy link
Copy Markdown
Owner

Mengapa

refresh-vuln-db.yml gagal 0 detik di SETIAP push (254 run gagal, event: push) padahal on:-nya cuma schedule + workflow_dispatch. gh run view mengonfirmasi: "This run likely failed because of a workflow file issue" — nama run muncul sebagai path file, bukan name: field → GitHub gagal meng-compile.

Root cause: step "Push branch + create PR" punya dua key env: (satu GH_TOKEN, satu blok display vars). PyYAML diam-diam ambil yang terakhir (jadi yaml.safe_load lolos), tapi parser GitHub Actions menolak duplicate mapping key → file invalid → run gagal di tiap push.

Bonus: step "Create branch + commit" punya blok env: yang ke-indent 10 spasi sehingga ketelan ke dalam run: | (var commit message jadi kosong).

Perbaikan

  • Merge dua blok env: di "Push branch + create PR" jadi satu (GH_TOKEN + 5 display vars).
  • Naikkan blok env: "Create branch + commit" ke step-level (8 spasi) sebelum run:.

Verifikasi

Recursive duplicate-key check + schema parse: OK, no duplicate keys. Kedua step sekarang punya env proper (dikonfirmasi via yaml.safe_load).

Setelah merge: workflow file valid → tidak ada lagi push-noise; schedule bulanan + dispatch berfungsi.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@sonarqubecloud

Copy link
Copy Markdown

@Wolfvin
Wolfvin merged commit bac242b into main Jul 13, 2026
1 of 7 checks passed
@Wolfvin
Wolfvin deleted the fix/refresh-vuln-db-workflow-invalid branch July 13, 2026 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant