Skip to content

Rebuild: attribute registered provider processes to threads - #105

Merged
Mjones13 merged 4 commits into
work/jones-rebuild/L16-goal-pausefrom
work/jones-rebuild/L17-process-attribution
Oct 5, 2026
Merged

Mjones13 merged 4 commits into
work/jones-rebuild/L16-goal-pausefrom
work/jones-rebuild/L17-process-attribution

Conversation

@Mjones13

@Mjones13 Mjones13 commented Oct 4, 2026 •

Copy link
Copy Markdown
Contributor

Port original owner PR #25 onto the nightly runtime so telemetry can attribute a registered Codex app-server process to its owning provider instance and thread. Registration occurs at process creation and is removed when its scope closes; generation tokens prevent an older registration from clearing a newer owner.

Stacked on #103. This layer preserves the original eight added cases and adds three lifecycle regressions. It also supplies the shared attribution layer to existing provider fixtures and declares the hydration dependency explicitly.

Validation: 505 tests passed across 21 affected modules on integrated commit 048eada82bc0ccd79e1653892aa326db171e89a1; contracts and server typechecks, scoped formatting/lint, and Knip passed. The inherited ACP publication repair is included. The integrated server typecheck and server bundle also passed.

No live provider or host-process qualification was performed. Ownership comes from the exact registered factory handle, never PID discovery; missing ownership is not permission to stop a process. PID reuse within the platform timestamp resolution remains a documented limit.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@Mjones13
Mjones13 merged commit f3b28f9 into work/jones-rebuild/L16-goal-pause Oct 5, 2026
18 checks passed
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