Skip to content

Stop syncing legacy model JSON files in sync-actions#145

Draft
Copilot wants to merge 2 commits into
mainfrom
copilot/remove-special-handling-models-json
Draft

Stop syncing legacy model JSON files in sync-actions#145
Copilot wants to merge 2 commits into
mainfrom
copilot/remove-special-handling-models-json

Conversation

Copilot AI commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

sync-actions.yml still treated .github/aw/models.json and .github/aw/model-multipliers.json as sync inputs, even though this repository only carries the compatibility matrix files. That produced avoidable warnings when syncing older gh-aw refs where those files do not exist.

  • Scope the sync to actual repo-owned files

    • Remove models.json and model-multipliers.json from the .github/aw cleanup and copy loop.
    • Keep sync behavior limited to:
      • .github/aw/compat.json
      • .github/aw/compat.schema.json
  • Align logging with synced artifacts

    • Update the post-sync listing to report only compat* files, so logs reflect the actual sync contract.
  • Align staging with synced artifacts

    • Remove the legacy model files from the guarded git add -A loop used before PR creation.
for f in compat.json compat.schema.json; do
  src="gh-aw/.github/aw/$f"
  dst="gh-aw-actions/.github/aw/$f"
  if [ -f "$src" ]; then
    cp "$src" "$dst"
  fi
done

GitHub Advanced Security started work on behalf of pelikhan June 9, 2026 14:48 View session
Copilot AI changed the title Remove special handling for legacy model sync files Stop syncing legacy model JSON files in sync-actions Jun 9, 2026
Copilot AI requested a review from pelikhan June 9, 2026 14:48
GitHub Advanced Security finished work on behalf of pelikhan June 9, 2026 14:49
@pelikhan

pelikhan commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

@copilot they are under the js folder

GitHub Advanced Security started work on behalf of pelikhan June 9, 2026 14:51 View session

Copilot AI commented Jun 9, 2026

Copy link
Copy Markdown
Contributor Author

@copilot they are under the js folder

Updated in 387cf15 — I clarified in sync-actions.yml that model data files are synced from actions/setup/js during the action-folder rsync step, not from .github/aw.

GitHub Advanced Security finished work on behalf of pelikhan June 9, 2026 14:52
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.

2 participants