Skip to content

fix: stabilize architecture sync and cloud batch tests#1100

Open
Serhan-Asad wants to merge 1 commit into
mainfrom
change/sync-main-20260519-152000
Open

fix: stabilize architecture sync and cloud batch tests#1100
Serhan-Asad wants to merge 1 commit into
mainfrom
change/sync-main-20260519-152000

Conversation

@Serhan-Asad
Copy link
Copy Markdown
Collaborator

Summary

  • Reconcile architecture metadata with current prompt files and fix stale include/dependency references.
  • Harden cloud batch submission against concurrent source tarball collisions and spot preemption retries.
  • Preserve 4xx response details for cloud insufficient-credit errors and expose failed cloud-regression command output.

Test plan

  • PDD_PATH=/private/tmp/pdd-sync-main-20260519-152000/pdd uv run pdd sync-architecture --dry-run
  • PDD_PATH=/private/tmp/pdd-sync-main-20260519-152000/pdd uv run pdd checkup --validate-arch-includes --project-root .
  • PDD_PATH=/private/tmp/pdd-sync-main-20260519-152000/pdd uv run --extra dev pytest tests/test_code_generator_main.py::test_full_gen_cloud_insufficient_credits_displays_balance tests/test_fix_main.py::test_fix_main_cloud_insufficient_credits_raises_error tests/test_fix_main.py::test_cloud_fix_errors_insufficient_credits tests/test_bug_main.py::test_bug_main_cloud_insufficient_credits_displays_balance tests/test_crash_main.py::test_crash_main_cloud_insufficient_credits_raises_error -q
  • python -m json.tool ci/cloud-batch/job-template.json >/dev/null && python -m json.tool ci/cloud-batch/job-template-standard.json >/dev/null && bash -n ci/cloud-batch/entrypoint.sh && bash -n ci/cloud-batch/submit.sh && bash -n tests/cloud_regression.sh && git diff --check
  • uv run --extra dev python -m build
  • make cloud-test: 77 passed, 0 failed, 0 errors

Copy link
Copy Markdown

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

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