Skip to content

fix(knowledge): preserve organization billing context in document worker - #7596

Merged
waleedlatif1 merged 1 commit into
stagingfrom
codex/organization-indexing-billing-context
Sep 8, 2026
Merged

fix(knowledge): preserve organization billing context in document worker#7596
waleedlatif1 merged 1 commit into
stagingfrom
codex/organization-indexing-billing-context

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • Preserve organization billing attribution when the document worker starts processing. The worker previously converted organization jobs into personal jobs, causing the processor to reject them.
  • Reuse the shared billing-context validator and cover organization preservation and mismatched ownership alongside the existing workspace and personal cases.

Type of Change

  • Bug fix

Testing

The new regression test fails before the fix. All 105 targeted processing, queue, and billing tests pass afterward. Repository lint, all 46 audits, block-registry validation, and docs-manifest validation pass.

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 8, 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 8, 2026 4:04pm UTC

Request Review

@waleedlatif1
waleedlatif1 merged commit 9b1d6ed into staging Sep 8, 2026
24 checks passed
@waleedlatif1
waleedlatif1 deleted the codex/organization-indexing-billing-context branch September 8, 2026 16:05
@greptile-apps

greptile-apps Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

The PR appears safe to merge and correctly preserves organization billing context without changing the established validation contract.

Summary

  • Replaces the two-branch manual reconstruction that converted organization jobs into personal jobs.
  • Adds regression coverage for organization context preservation and mismatched organization attribution.
  • Retains existing behavior for workspace and personal processing contexts.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  Queue[Queued document payload] --> Validate[Validate processing payload]
  Validate --> Context[Derive shared billing context]
  Context --> Personal[Personal context]
  Context --> Workspace[Workspace attribution]
  Context --> Organization[Organization attribution]
  Personal --> Processor[Document processor]
  Workspace --> Processor
  Organization --> Processor
Loading

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