Skip to content

Fix call artifact identity resolution#2068

Merged
dionysuzx merged 2 commits into
masterfrom
codex/fix-p2p-artifact-ingestion
May 20, 2026
Merged

Fix call artifact identity resolution#2068
dionysuzx merged 2 commits into
masterfrom
codex/fix-p2p-artifact-ingestion

Conversation

@dionysuzx
Copy link
Copy Markdown
Contributor

@dionysuzx dionysuzx commented May 20, 2026

Summary

  • resolve asset pipeline targets from mapped call identity before downloading, summarizing, or publishing artifacts
  • remove the unsafe Zoom date/topic fallback that could ingest a recording from another series
  • fail on ambiguous call numbers or same-day recordings instead of guessing
  • reject numbered artifact directories that do not match a mapped occurrence during manifest generation
  • remove the invalid P2P artifact and orphan PQ Interop artifacts generated by the previous fallback

Why

The pipeline treated date plus inferred topic prefix as enough to recover from missing Zoom mappings. That made an incorrect recording look valid and allowed it into the manifest.

The pipeline now requires a mapped occurrence identity for numbered artifacts and uses existing artifact directories only after the target call is known.

Validation

  • .github/ACDbot/.venv/bin/python -m pytest .github/ACDbot/tests
  • .github/ACDbot/.venv/bin/python .github/ACDbot/scripts/asset_pipeline/generate_manifest.py
  • GitHub checks passed: validate, GitGuardian Security Checks

@dionysuzx dionysuzx force-pushed the codex/fix-p2p-artifact-ingestion branch 8 times, most recently from 780c48b to a1c3a9a Compare May 20, 2026 20:50
@dionysuzx dionysuzx force-pushed the codex/fix-p2p-artifact-ingestion branch from a1c3a9a to 4d6a798 Compare May 20, 2026 21:06
@dionysuzx dionysuzx changed the title Fix P2P artifact ingestion Fix call artifact identity resolution May 20, 2026
@dionysuzx dionysuzx marked this pull request as ready for review May 20, 2026 21:21
@dionysuzx dionysuzx merged commit 8472394 into master May 20, 2026
2 checks passed
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