Skip to content

feat(OMN-14058): add optional tenant_id to delegation wire DTOs [OPERATOR-ACCEPTED INTERIM] #1404

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
jonahgabriel merged 1 commit into from
Jul 8, 2026
Merged

feat(OMN-14058): add optional tenant_id to delegation wire DTOs [OPERATOR-ACCEPTED INTERIM] #1404

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 12 additions & 0 deletions src/omnibase_core/models/delegation/wire/model_delegation_result.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -134,6 +134,18 @@ class ModelDelegationResult(BaseModel):
"(e.g. a remote-agent A2A terminal). OMN-13649."
),
)
# string-id-ok: tenant_id is a named tenant identifier, not a UUID
tenant_id: str | None = Field(
default=None,
description=(
"Multi-tenant isolation identifier carried from the originating "
"request (or the ONEX_TENANT_ID env-var fallback at "
"request-acceptance). OPERATOR-ACCEPTED INTERIM (OMN-14058): None "
"means no tenant identity was resolved and the delegation "
"projection falls back to the shared 'omninode' tenant column "
"default. The durable per-tenant identity design is OMN-14107."
),
)

@model_validator(mode="after")
def validate_total_tokens(self) -> Self:
Expand Down
11 changes: 11 additions & 0 deletions src/omnibase_core/models/delegation/wire/model_delegation_wire_request.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -159,6 +159,17 @@ class ModelDelegationRequest(BaseModel):
default=(),
description="Request-level quality checks enforced by the quality gate.",
)
# string-id-ok: tenant_id is a named tenant identifier, not a UUID
tenant_id: str | None = Field(
default=None,
description=(
"Multi-tenant isolation identifier. OPERATOR-ACCEPTED INTERIM "
"(OMN-14058): when unset, the orchestrator falls back to the "
"ONEX_TENANT_ID env var at request-acceptance so delegation "
"projections stop defaulting to the shared 'omninode' tenant. "
"The durable per-tenant identity design is OMN-14107."
),
)

@model_validator(mode="after")
def _validate_compliance_loop_config(self) -> Self:
Expand Down
Loading