Skip to content

fix: snapshot published runtime artifacts#22

Merged
1aal merged 1 commit into
apecloud-basefrom
fix/runtime-published-artifact-snapshots
Jun 11, 2026
Merged

fix: snapshot published runtime artifacts#22
1aal merged 1 commit into
apecloud-basefrom
fix/runtime-published-artifact-snapshots

Conversation

@1aal

@1aal 1aal commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • publish report artifacts by stable relative file path instead of content digest so an appending/re-written report appears as one asset
  • copy the latest discovered content into .published/<artifactId>/ before advertising metadata to Cloud
  • serve artifact IDs from .published snapshots before falling back to the legacy live-directory scan, and skip .published during discovery

Behavior

  • A file modified multiple times in one run keeps one artifact ID/card and serves the latest published content.
  • Different report files still become different artifacts.
  • The legacy live scan remains as fallback for existing links and old Runtime data.

Verification

  • uv run pytest tests/runtime_manager/test_report_artifacts.py
  • uv run pytest tests/runtime_manager
  • uv run ruff check runtime_manager/artifacts.py tests/runtime_manager/test_report_artifacts.py
  • python3 -m py_compile runtime_manager/artifacts.py
  • git diff --check
  • git diff --check origin/apecloud-base...HEAD

@1aal 1aal force-pushed the fix/runtime-published-artifact-snapshots branch from fd3156b to 4705587 Compare June 11, 2026 08:25
@1aal 1aal marked this pull request as ready for review June 11, 2026 08:32
@1aal 1aal merged commit 88c8c78 into apecloud-base Jun 11, 2026
7 checks passed
@1aal 1aal deleted the fix/runtime-published-artifact-snapshots branch June 11, 2026 08:32
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