build_image.py push-first strategy: The push_image function deleted the existing manifest before pushing (Gitea #31964 workaround). When push failed for other reasons (HTTP 500), the old tag was lost, breaking all CI jobs using that image. Now tries push first, only deletes+retries on "already exists" error.
Remove cross-repo contract tests: devx tests were validating infra, grm, sso-bridge, and mattermost-oidc files. This is an architecture violation — devx must not be aware of other repos. Tests now only validate devx's own workflows and skills.
Closes DEVX-159
## Summary
Two fixes:
1. **build_image.py push-first strategy**: The `push_image` function deleted the existing manifest before pushing (Gitea #31964 workaround). When push failed for other reasons (HTTP 500), the old tag was lost, breaking all CI jobs using that image. Now tries push first, only deletes+retries on "already exists" error.
2. **Remove cross-repo contract tests**: devx tests were validating infra, grm, sso-bridge, and mattermost-oidc files. This is an architecture violation — devx must not be aware of other repos. Tests now only validate devx's own workflows and skills.
Closes DEVX-159
kireto
approved these changes 2026-08-26 07:25:46 +00:00
The push_image function deleted the existing manifest before pushing
(Gitea #31964 workaround). When push failed for other reasons (HTTP
500), the old tag was lost, breaking all CI jobs using that image.
Now tries push first, only deletes+retries on "already exists" error.
Closes DEVX-159
Generated with [Devin](https://devin.ai)
Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
kireto
Summary
Two fixes:
build_image.py push-first strategy: The
push_imagefunction deleted the existing manifest before pushing (Gitea #31964 workaround). When push failed for other reasons (HTTP 500), the old tag was lost, breaking all CI jobs using that image. Now tries push first, only deletes+retries on "already exists" error.Remove cross-repo contract tests: devx tests were validating infra, grm, sso-bridge, and mattermost-oidc files. This is an architecture violation — devx must not be aware of other repos. Tests now only validate devx's own workflows and skills.
Closes DEVX-159
LGTM — push-first strategy + architecture fix.
b9bc97be63to9d7b01a468