Skip to content

fix(provenance): preserve execution files across durable surfaces - #7778

Merged
icecrasher321 merged 2 commits into
stagingfrom
codex/fix-execution-file-provenance
Sep 11, 2026
Merged

icecrasher321 merged 2 commits into
stagingfrom
codex/fix-execution-file-provenance

Conversation

@icecrasher321

@icecrasher321 icecrasher321 commented Sep 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Preserve provenance from Function exports through file reads, parsing, copies, archive extraction, and KB ingestion. Bind sidecars to canonical content revisions and retain legacy compatibility.
  • Carry rendered image and document contributors through file conversions and Function inputs; prevent archived metadata or concurrent reads from dropping lineage.
  • Recognize owned Azure execution URLs using the configured storage client endpoint. Keep model and Function admission at their consuming boundaries.
  • Keep existing feature flags and use the current file sidecar schema.

Type of Change

  • Bug fix

Testing

  • Real PostgreSQL/Redis integration covers execution ZIP → workspace extraction → table rows → immutable KB copy → delayed indexing/search, plus unknown, secret-bearing, legacy, archived, and wrong-run cases. External embeddings are mocked.
  • Focused provenance, Function, file, table, memory, and KB suites; 350 regression tests for the review follow-up; 11 real database integration scenarios; type checks; lint and all 46 audits, including API boundary validation. Bun 1.4.1.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Sep 11, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated
docs Skipped Skipped Sep 11, 2026 7:03pm UTC

Request Review

@icecrasher321
icecrasher321 force-pushed the codex/fix-execution-file-provenance branch from 8159b75 to c5dc6a7 Compare September 11, 2026 18:47
@greptile-apps

greptile-apps Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The reviewed provenance changes appear safe to merge, with no outstanding blocking finding.

Summary

  • Binds sidecars to canonical file identities and content revisions.
  • Carries source and rendered contributor identities through Function reads and mounts.
  • Classifies harvested execution files and preserves provenance through durable file surfaces.
  • Recognizes Azure execution URLs from the configured Blob client endpoint.
  • Adds focused unit and integration coverage for legacy, unknown, archived, concurrent, and cross-scope cases.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  A[Function or file operation] --> B[Authorize canonical stored file]
  B --> C[Read bytes and contributor identities]
  C --> D{Rendered content?}
  D -->|Yes| E[Apply opaque egress admission]
  D -->|No| F[Import runtime provenance]
  E --> F
  F --> G[Execute or transform]
  G --> H[Classify output provenance]
  H --> I[Persist bytes and revision-bound sidecar]
  I --> J[Copy, extract, or ingest into durable surface]
Loading

Reviews (2) · Last reviewed commit: "fix(provenance): preserve storage identi..."

Comment thread apps/sim/lib/internal/file/parser.ts Outdated
@icecrasher321
icecrasher321 force-pushed the codex/fix-execution-file-provenance branch from c5dc6a7 to f6b93b6 Compare September 11, 2026 19:03
@icecrasher321

Copy link
Copy Markdown
Collaborator Author

@greptile

@icecrasher321

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@icecrasher321 I have started the AI code review. It will take a few minutes to complete.

@icecrasher321
icecrasher321 merged commit e21b9a6 into staging Sep 11, 2026
34 checks passed
@icecrasher321
icecrasher321 deleted the codex/fix-execution-file-provenance branch September 11, 2026 19:09
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