Skip to content

Ingest reusable sweeps with normal artifact handling / 使用常规产物处理摄取复用扫描#582

Merged
cquil11 merged 3 commits into
masterfrom
agent/direct-reuse-ingest
Jul 16, 2026
Merged

Ingest reusable sweeps with normal artifact handling / 使用常规产物处理摄取复用扫描#582
cquil11 merged 3 commits into
masterfrom
agent/direct-reuse-ingest

Conversation

@cquil11

@cquil11 cquil11 commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Summary

  • accept source-run-id and merge-run-id in both ingest workflows
  • download the source run exactly like normal ingestion: ignore expired uploads and keep the newest upload for each exact artifact name
  • replace the source changelog-metadata with the newest merge-run changelog
  • write source provenance, then hand the directory to the existing database ingestion code
  • remove benchmark-content inspection, logical-point inference, companion pairing, and reusable-artifact validation

End-to-end flow

  1. InferenceX sends the benchmark source run ID and the main publication run ID.
  2. InferenceX-app lists both runs and selects source artifacts using exact-name deduplication only.
  3. It substitutes the merge-run changelog and records source provenance.
  4. The existing ingestion code maps rows, skips failed benchmark rows, and writes the database exactly as it does for normal ingestion.

Validation

  • 362 database unit tests passed
  • TypeScript typecheck, lint, formatting, and diff checks passed
  • PR #2145 source run 29367944489: dry run selected 262 artifacts from 288 uploads
  • PR #2224 source run 29445892486: dry run selected 85 artifacts from 86 uploads
  • both dry runs used merge run 29508236865 for changelog metadata and downloaded no artifacts

Related producer PR: SemiAnalysisAI/InferenceX#2244

中文说明

  • 两个摄取工作流均接收 source-run-idmerge-run-id
  • 源运行产物沿用常规摄取方式:忽略已过期上传,并按精确产物名称保留最新上传
  • 使用合并运行中最新的 changelog-metadata 替换源运行变更日志
  • 写入源运行溯源信息后,将产物目录交给现有数据库摄取代码
  • 删除基准测试内容检查、逻辑点推断、配套产物匹配和复用产物验证

端到端流程现在只有:InferenceX 发送两个运行 ID,InferenceX-app 下载常规源产物并替换合并运行变更日志,随后由现有摄取逻辑映射数据、跳过失败基准测试结果并写入数据库。

验证结果:362 个数据库单元测试通过;类型检查、lint、格式检查和差异检查通过;PR #2145 与 PR #2224 的实时 dry run 均成功,且未下载任何产物。

Centralize CI artifact preparation in TypeScript. Select the newest valid artifact family per logical benchmark point, use publication-run changelog metadata, discover attempts from GitHub, preserve source provenance, and fail closed on preparation errors.

将 CI 工件准备集中到 TypeScript。按每个逻辑基准点选择最新有效的工件组,使用发布运行的变更日志元数据,从 GitHub 自动获取尝试次数,保留源运行来源,并在准备错误时安全失败。
@cquil11 cquil11 requested a review from adibarra as a code owner July 16, 2026 16:38
@vercel

vercel Bot commented Jul 16, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
inferencemax-app Ready Ready Preview, Comment Jul 16, 2026 5:14pm

Request Review

Pass the workspace artifact path through the step environment before using it in validator shell commands, satisfying the GitHub Actions security audit without changing behavior.

先通过步骤环境变量传递工作区工件路径,再在验证器的 shell 命令中使用,以在不改变行为的情况下通过 GitHub Actions 安全审计。
Remove benchmark-content inspection, logical-point inference, companion pairing, and reusable-set validation. Keep only exact-name deduplication and merge-run changelog substitution before the existing ingestion path.\n\n中文:移除基准测试内容检查、逻辑点推断、配套产物匹配和复用产物集验证。仅保留按精确名称去重及合并运行变更日志替换,然后交给现有摄取流程处理。
@cquil11 cquil11 changed the title Ingest reusable sweeps directly from source runs Ingest reusable sweeps with normal artifact handling / 使用常规产物处理摄取复用扫描 Jul 16, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 9adcb24. Configure here.

Comment thread .github/workflows/ingest-agentic-results.yml
@cquil11 cquil11 merged commit 93c255c into master Jul 16, 2026
27 checks passed
@cquil11 cquil11 deleted the agent/direct-reuse-ingest branch July 16, 2026 17:22
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