Merge pull request #95 from phj1081/codex/owner/ejclaw
Remove paired execution shared shim
This commit is contained in:
@@ -45,7 +45,7 @@ There are also several signs that the code is already moving in the right direct
|
|||||||
|
|
||||||
- `src/message-runtime-*` is already split by responsibility.
|
- `src/message-runtime-*` is already split by responsibility.
|
||||||
- `src/provider-retry.ts` is a good example of extracting real duplication.
|
- `src/provider-retry.ts` is a good example of extracting real duplication.
|
||||||
- `src/paired-execution-context-shared.ts` is already acting as a central status-transition guard.
|
- `src/paired-task-status.ts` is already acting as a central status-transition guard.
|
||||||
- `setup/register.ts` is already much simpler than older compatibility assumptions would suggest.
|
- `setup/register.ts` is already much simpler than older compatibility assumptions would suggest.
|
||||||
|
|
||||||
So the right move is **targeted simplification**, not broad reorganization.
|
So the right move is **targeted simplification**, not broad reorganization.
|
||||||
@@ -533,7 +533,7 @@ This is not a request to rewrite all domain types. It is a request to stop overl
|
|||||||
|
|
||||||
### Status transition rule
|
### Status transition rule
|
||||||
|
|
||||||
All paired-task status writes should continue converging on `src/paired-execution-context-shared.ts`. Do not allow status mutation rules to scatter again.
|
All paired-task status writes should continue converging on `src/paired-task-status.ts`. Do not allow status mutation rules to scatter again.
|
||||||
|
|
||||||
### Acceptance criteria
|
### Acceptance criteria
|
||||||
|
|
||||||
|
|||||||
@@ -1,27 +0,0 @@
|
|||||||
export { requestArbiterOrEscalate } from './paired-arbiter-request.js';
|
|
||||||
export type { RequestArbiterOrEscalateFn } from './paired-arbiter-request.js';
|
|
||||||
export {
|
|
||||||
resolveOwnerCompletionSignal,
|
|
||||||
resolveReviewerCompletionSignal,
|
|
||||||
resolveReviewerFailureSignal,
|
|
||||||
} from './paired-completion-signals.js';
|
|
||||||
export type { CompletionSignal } from './paired-completion-signals.js';
|
|
||||||
export {
|
|
||||||
classifyArbiterVerdict,
|
|
||||||
parseVisibleVerdict,
|
|
||||||
} from './paired-verdict.js';
|
|
||||||
export type { ArbiterVerdictResult, VisibleVerdict } from './paired-verdict.js';
|
|
||||||
export {
|
|
||||||
hasCodeChangesSinceRef,
|
|
||||||
resolveCanonicalSourceRef,
|
|
||||||
} from './paired-source-ref.js';
|
|
||||||
export {
|
|
||||||
ALLOWED_PAIRED_STATUS_TRANSITIONS,
|
|
||||||
applyPairedTaskPatch,
|
|
||||||
assertPairedTaskStatusTransition,
|
|
||||||
transitionPairedTaskStatus,
|
|
||||||
} from './paired-task-status.js';
|
|
||||||
export type {
|
|
||||||
ApplyPairedTaskPatchFn,
|
|
||||||
TransitionPairedTaskStatusFn,
|
|
||||||
} from './paired-task-status.js';
|
|
||||||
Reference in New Issue
Block a user