Skip to content

ci: vendor validation scripts and remove remote action pins - #47

Merged
hyperpolymath merged 1 commit into
mainfrom
fix-ci-actions
Jul 27, 2026
Merged

ci: vendor validation scripts and remove remote action pins#47
hyperpolymath merged 1 commit into
mainfrom
fix-ci-actions

Conversation

@hyperpolymath

@hyperpolymath hyperpolymath commented Jul 27, 2026

Copy link
Copy Markdown
Owner

Automated PR to fix CI after deleted actions.


Summary by Gitar

  • CI Enhancements:
    • Added vendored validation script .githooks/validate-a2ml.sh for A2ML manifest checking
    • Added vendored validation script .githooks/validate-k9.sh for K9 configuration checking
    • Updated .github/workflows/dogfood-gate.yml to execute local validation scripts instead of remote actions

This will update automatically on new commits.

@gitar-bot

gitar-bot Bot commented Jul 27, 2026

Copy link
Copy Markdown
CI failed: CI failures occurred in validation jobs due to missing required identity fields in A2ML manifests and missing magic numbers/pedigree fields in K9 contract files.

Overview

Two validation jobs failed during the CI run: A2ML manifest validation failed due to missing identity fields, and K9 contract validation failed due to missing magic numbers and pedigree metadata in contract files.

Failures

A2ML Manifest Validation Failure (confidence: high)

  • Type: build
  • Affected jobs: 90126178342
  • Related to change: yes
  • Root cause: The A2ML validation script encountered an error due to a missing required identity field (agent-id, name, or project) in scanned manifest files.
  • Suggested fix: Inspect the .a2ml manifest files and ensure all required identity fields are properly specified.

K9 Contract Validation Failure (confidence: high)

  • Type: test
  • Affected jobs: 90126178257
  • Related to change: yes
  • Root cause: Contract files (such as template-hunt.k9.ncl) failed validation because they lacked the required 'K9!' magic number on the first non-empty line and were missing the 'name' field in the pedigree block.
  • Suggested fix: Update the affected K9 contract files to include 'K9!' as the first non-empty line and provide a valid 'name' field in the pedigree block.

Summary

  • Change-related failures: 2 validation failures (A2ML manifest and K9 contract checks failing due to content requirements)
  • Infrastructure/flaky failures: None
  • Recommended action: Update the manifest and contract files in the repository to satisfy the validation script requirements.
Code Review ✅ Approved

Vendors validation scripts into .githooks and updates the dogfood-gate workflow to run them locally instead of relying on remote actions. No issues found.

Auto-approved: No blocking issues found.
Please see Auto-approve Docs for details on setting custom approval criteria.

Tip

Comment Gitar fix CI or enable auto-apply: gitar auto-apply:on

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@gitar-bot gitar-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gitar has auto-approved this PR (configure)

@hyperpolymath
hyperpolymath merged commit 3fd4dfe into main Jul 27, 2026
30 of 33 checks passed
@hyperpolymath
hyperpolymath deleted the fix-ci-actions branch July 27, 2026 23:26
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