Skip to content

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

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

ci: vendor validation scripts and remove remote action pins#55
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 Workflows:
    • Vendored validation scripts for A2ML and K9 manifests into .githooks/
    • Replaced remote ecosystem validation actions with local script executions in dogfood-gate.yml

This will update automatically on new commits.

@gitar-bot

gitar-bot Bot commented Jul 27, 2026

Copy link
Copy Markdown
CI failed: 3 validation and code generation failures occurred due to uncommitted golden sample updates, missing identity fields in A2ML manifests, and incorrect K9 contract headers.

Overview

Three separate validation and test jobs failed during CI: A2ML manifest validation failed due to missing identity fields, codegen verification failed because golden sample files were out of date with the latest code generator output, and K9 contract validation failed due to missing magic numbers and pedigree metadata.

Failures

A2ML Manifest Validation Failure (confidence: high)

  • Type: test
  • Affected jobs: 90125543650
  • Related to change: yes
  • Root cause: The script .githooks/validate-a2ml.sh found 1 validation error during the scan of .a2ml files: Missing required identity field (agent-id, name, or project) in the manifest files.
  • Suggested fix: Inspect the .a2ml manifest files to ensure all required identity fields like agent-id, name, or project are properly populated, then re-run the validation script.

Codegen Golden Sample Mismatch (confidence: high)

  • Type: test
  • Affected jobs: 90125543772
  • Related to change: yes
  • Root cause: The cargo generator produced newly updated Chapel wrapper code with strict storage failure checking, but the golden reference files under examples/golden/generated were not updated to match, causing diff to fail.
  • Suggested fix: Run the code generator locally (cargo run -- generate -m examples/golden/echo.toml -o examples/golden/generated) to update the golden sample files, then commit the updated files.

K9 Contract Validation Failure (confidence: high)

  • Type: test
  • Affected jobs: 90125543575
  • Related to change: yes
  • Root cause: The validation script .githooks/validate-k9.sh found 2 errors: missing 'K9!' magic number on the first non-empty line and missing 'name' field in the pedigree block.
  • Suggested fix: Update the K9 contract files to include the 'K9!' magic number on the first non-empty line and add the missing 'name' field to their pedigree blocks.

Summary

  • Change-related failures: 3 (manifest validation, golden sample out-of-date, and K9 contract formatting issues)
  • Infrastructure/flaky failures: 0
  • Recommended action: Update the A2ML manifest identity fields, regenerate and commit the golden samples, and fix the K9 contract headers and pedigree metadata before retrying CI.
Code Review ✅ Approved

Vendors manifest validation scripts and transitions CI workflows to execute them locally, removing reliance on remote action pins. 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 783e731 into main Jul 27, 2026
33 of 37 checks passed
@hyperpolymath
hyperpolymath deleted the fix-ci-actions branch July 27, 2026 22:49
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