Skip to content

feat: Scope SDK identity and scratch paths per request - #80

Closed
CyMule wants to merge 6 commits into
mainfrom
feat/invocation-scope-helpers
Closed

CyMule wants to merge 6 commits into
mainfrom
feat/invocation-scope-helpers

Conversation

@CyMule

@CyMule CyMule commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

The SDK exposes current_invocation_identity() and invocation_scratch_dir() for request-scoped plugin identity and scratch paths. This lets a long-lived plugin process serve concurrent invocations while keeping their files and attribution separate.

A bound invocation context is authoritative, including fields it leaves absent. Without a context, identity falls back to the process environment and the scratch root is unchanged. With a context, the helper returns a sanitized job/node/invocation path.

Validation: 211 SDK tests and Ruff, covering concurrent calls, safe paths, batch correlation and streaming context retention.

Package metadata and changelog specify stable 0.1.1. After merge, publish a GitHub release tagged 0.1.1 from the merged commit; .github/workflows/release.yml builds and publishes the package to PyPI.

@CyMule CyMule changed the title feat: invocation identity and per-invocation scratch helpers feat(invocation): Scope identity and scratch paths per request Oct 19, 2026
@CyMule

CyMule commented Oct 19, 2026

Copy link
Copy Markdown
Contributor Author

@cubic-dev-ai review

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 19, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review

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

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 4 files

Shadow auto-approve: would not auto-approve because issues were found.

View guided diff | Re-trigger cubic

@CyMule

CyMule commented Oct 19, 2026

Copy link
Copy Markdown
Contributor Author

@cubic-dev-ai review

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 19, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review

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

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 4 files

Shadow auto-approve: would not auto-approve. This PR does not meet the repository auto-approval settings.

View guided diff | Re-trigger cubic

@CyMule CyMule changed the title feat(invocation): Scope identity and scratch paths per request feat: Scope SDK identity and scratch paths per request Oct 19, 2026
@CyMule
CyMule marked this pull request as ready for review October 19, 2026 01:38

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 4 files

Shadow auto-approve: would not auto-approve. This PR does not meet the repository auto-approval settings.

View guided diff | Re-trigger cubic

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

0 issues found across 2 files (changes from recent commits).

Shadow auto-approve: would not auto-approve. This PR does not meet the repository auto-approval settings.

View guided diff | Re-trigger cubic

@CyMule

CyMule commented Oct 19, 2026

Copy link
Copy Markdown
Contributor Author

Closing without merging. These request-scoped identity and scratch-directory helpers won't be added to this package, since its request-scoped helpers are being retired rather than extended.

@CyMule CyMule closed this Oct 19, 2026
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