Skip to content

feat(ci): attach test and repro log artifacts to PR orchestrator runs#261

Closed
djm81 wants to merge 1 commit intodevfrom
feature/ci-01-pr-orchestrator-log-artifacts
Closed

feat(ci): attach test and repro log artifacts to PR orchestrator runs#261
djm81 wants to merge 1 commit intodevfrom
feature/ci-01-pr-orchestrator-log-artifacts

Conversation

@djm81
Copy link
Copy Markdown
Collaborator

@djm81 djm81 commented Feb 16, 2026

Description

Attach test and repro log artifacts to PR Orchestrator workflow runs so failed CI can be debugged by downloading full logs from the Actions Artifacts section without re-running locally.

  • Tests job: Runs hatch run smart-test-full and uploads logs/tests/ as test-logs artifact; coverage XML still uploaded for quality gates.
  • Contract-first-ci job: Captures specfact repro stdout/stderr to logs/repro/, uploads repro-logs and repro-reports (.specfact/reports/enforcement/) as artifacts.
  • Docs: New "CI and GitHub Actions" section in Troubleshooting describing artifact names and how to use them.

Fixes #260

New Features (none)

Contract References: None (workflow and docs only). OpenSpec change: ci-01-pr-orchestrator-log-artifacts.

Type of Change

  • ✨ New feature (non-breaking change which adds functionality)
  • 📚 Documentation update

Contract-First Testing Evidence

No application code or public API changes; workflow YAML and docs only. Contract/test evidence N/A.

  • Workflow lint: hatch run lint-workflows
  • Contract validation: hatch run contract-test ✅ (cached)
  • YAML lint: hatch run yaml-lint

How Has This Been Tested?

  • Workflow syntax validated with hatch run lint-workflows.
  • Contract-test and smart-test-unit run (no modified Python files; no new tests required for workflow-only change).

Checklist

  • My code follows the style guidelines
  • I have performed a self-review
  • I have made corresponding changes to documentation
  • Version and CHANGELOG updated (0.31.1)

Quality Gates Status

  • Workflow lint
  • YAML lint
  • Contract validation

- Tests job: run smart-test-full, upload logs/tests/ as test-logs artifact
- Contract-first-ci: capture repro to logs/repro/, upload repro-logs and repro-reports
- Docs: CI and GitHub Actions section in troubleshooting (artifact names, usage)
- Version 0.31.1, CHANGELOG entry

Implements OpenSpec change ci-01-pr-orchestrator-log-artifacts. Fixes #260.

Co-authored-by: Cursor <cursoragent@cursor.com>
@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.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@github-actions
Copy link
Copy Markdown
Contributor

SpecFact CLI Validation Report

All validations passed!
Duration: 19.66s
Checks: 4 total (2 passed) (2 skipped)

@djm81
Copy link
Copy Markdown
Collaborator Author

djm81 commented Feb 16, 2026

Dupicate.

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