Skip to content

Expose les outputs de l'action de notification#14

Merged
kdelfour merged 1 commit into
mainfrom
claudettes/issue-1
May 11, 2026
Merged

Expose les outputs de l'action de notification#14
kdelfour merged 1 commit into
mainfrom
claudettes/issue-1

Conversation

@claudettes-bot

Copy link
Copy Markdown

Résumé

  • Ajout d'une section outputs dans action.yml déclarant sent, status, error et attempts
  • Modification de sendStoatWebhook() pour retourner le nombre de tentatives HTTP
  • Implémentation d'une fonction setOutput() pour publier les outputs via GITHUB_OUTPUT
  • Ajout de l'input optionnel dry_run (booléen, défaut false)
  • Sanitisation des messages d'erreur : suppression de la valeur du webhook_url
  • Mise à jour du README.md avec tableau des outputs et exemple de workflow
  • Régénération de dist/index.js via pnpm build

Critères d'acceptation

  • Le fichier action.yml contient une section outputs avec sent, status, error et attempts
  • Un workflow consommateur peut lire ${{ steps.notify.outputs.sent }} et ${{ steps.notify.outputs.status }}
  • sendStoatWebhook() retourne { attempts: number } avec les valeurs attendues (1, 2, ou 0)
  • Les valeurs sensibles (webhook_url) sont absentes des outputs, y compris du champ error
  • L'input optionnel dry_run produit status=dry_run sans appel HTTP
  • pnpm typecheck, pnpm test et pnpm build passent sans erreur
  • dist/index.js est régénéré et inclus dans le commit
  • Le README.md contient le tableau des outputs avec exemple d'utilisation

Closes #1

Add `sent`, `status`, `error`, and `attempts` outputs so consumer
workflows can branch on the notification result without parsing logs.
Introduce a `dry_run` input that publishes `status=dry_run` without
issuing an HTTP request, and route `event: auto` to `status=skipped`
when no template maps the event. Sanitize the `webhook_url` (and any
URL leaked in server error bodies) from the `error` output before
writing to `GITHUB_OUTPUT`.
@kdelfour
kdelfour merged commit f69242d into main May 11, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

01 - Expose action outputs for delivery status

1 participant