Skip to content

fix(run-once): report untrusted artifact authors#80

Merged
rochecompaan merged 1 commit into
mainfrom
fix/artifact-author-diagnostics
Jul 11, 2026
Merged

fix(run-once): report untrusted artifact authors#80
rochecompaan merged 1 commit into
mainfrom
fix/artifact-author-diagnostics

Conversation

@rochecompaan

Copy link
Copy Markdown
Owner

Summary

  • trust both configured and default Forgejo tea users for deterministic artifact comments
  • emit a terminal-only warning when deterministic spec/plan artifacts are skipped for an untrusted author
  • strip username-bearing console messages from JSONL run logs

Verification

  • node --test src/cli/commands/run-once/console-progress.test.ts src/cli/commands/run-once/progress.test.ts src/cli/commands/run-once/artifact-source-stage.test.ts
  • node --test src/workflow/artifacts/published-artifacts.test.ts src/cli/commands/run-once/artifact-source-stage.test.ts src/cli/commands/run-once/artifact-sources.test.ts src/cli/commands/run-once/console-progress.test.ts src/cli/commands/run-once/progress.test.ts src/host/forgejo-tea.test.ts
  • npm test
  • npm run lint:ts
  • npx prettier --check src/cli/commands/run-once/artifact-source-stage.ts src/cli/commands/run-once/artifact-source-stage.test.ts src/cli/commands/run-once/console-progress.ts src/cli/commands/run-once/console-progress.test.ts src/cli/commands/run-once/pipeline.ts src/cli/commands/run-once/progress.ts src/cli/commands/run-once/progress.test.ts src/workflow/artifacts/published-artifacts.ts src/host/forgejo-tea.ts src/host/forgejo-tea.test.ts

Trust the configured and default Forgejo tea users for artifact comments. Emit terminal-only diagnostics when deterministic artifacts are skipped because their author is untrusted, while keeping usernames out of JSONL run logs.
@rochecompaan rochecompaan merged commit c72afbd into main Jul 11, 2026
2 checks passed
@rochecompaan rochecompaan deleted the fix/artifact-author-diagnostics branch July 11, 2026 08:47
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