fsolla/teqo-backup
1
0
Fork 0
mirror of https://github.com/fsolla/teqo.git synced 2026-11-13 06:43:06 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix(api): removeLabels idempotente — 404 em label ausente não derruba o flip (dispatch re-run 404'd no in-progress já removido)

Browse source
This commit is contained in:
Francisco Solla 2026-08-23 15:13:48 -03:00
parent b00c69c9d8
commit 4486c05b49
1 changed files with 9 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
scripts/lib/github-api.mjs
View file

@ -88,7 +88,7 @@ export const createApi = ({
}
}
const request = async (path, { method = 'GET', body, query } = {}) => {
const request = async (path, { method = 'GET', body, query, ignoreNotFound = false } = {}) => {
const qs = query
? '?' +
Object.entries(query)
@ -131,6 +131,7 @@ export const createApi = ({
}
if (response.status === 404 && method === 'GET') return null
if (!response.ok) {
if (ignoreNotFound && response.status === 404) return null
throw new Error(`GitHub API ${method} ${path} → ${response.status}: ${text.slice(0, 400)}`)
}
if (!text) return null
@ -237,13 +238,19 @@ export const createApi = ({
body: { labels },
}),
/** Removes labels by NAME (GitHub DELETE is by label name). */
/**
* Removes labels by NAME (GitHub DELETE is by label name). Idempotent:
* removing an already-absent label (404) is the desired end state — a
* re-run of the flip/recovery path must not fail (live finding OPS46-S1,
* dispatch re-run after a manual flip 404'd on `in-progress`).
*/
removeLabels: async (number, labels) => {
for (const label of labels) {
await request(
`/repos/${owner}/${name}/issues/${number}/labels/${encodeURIComponent(label)}`,
{
method: 'DELETE',
ignoreNotFound: true,
},
)
}