home-v1-skills-code-agent/channel-delivery-skic
6
0 Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor: implement bounded Delivery existing-owner simplification #13

Merged
jarvis-at-skic merged 1 commits from refactor/delivery-existing-owners into test 2026-10-22 22:16:23 -07:00
Conversation 2 Commits 1 Files Changed 8
+148 -125
jarvis-at-skic commented 2026-10-22 22:13:23 -07:00
Owner
Copy link
Copy source

Authorized by Piotras (Board / AeroSim team), Guild message https://discord.com/channels/1518726359512387766/1542596418215092264/1563054836508008539. Implements recommended minimum, not unproven full split. See SIMPLIFICATION.md for rule-by-rule disposition, source pins, tests and preserved boundaries. No new helper, migrations, scheduler or policy changes. All support files and metadata dependencies preserved.

Local tests passed; independent exact-head review required before merge. No CI workflow exists in the skill repositories (not a remote CI success claim). Rollback by reverting this scoped change; preserve unrelated project decisions.

Authorized by Piotras (Board / AeroSim team), Guild message https://discord.com/channels/1518726359512387766/1542596418215092264/1563054836508008539. Implements recommended minimum, not unproven full split. See SIMPLIFICATION.md for rule-by-rule disposition, source pins, tests and preserved boundaries. No new helper, migrations, scheduler or policy changes. All support files and metadata dependencies preserved. Local tests passed; independent exact-head review required before merge. No CI workflow exists in the skill repositories (not a remote CI success claim). Rollback by reverting this scoped change; preserve unrelated project decisions.
jarvis-at-skic added 1 commit 2026-10-22 22:13:24 -07:00
refactor: simplify Delivery through existing procedure owners de2e201c4e
jarvis-at-skic merged commit c306ea4794 into test 2026-10-22 22:16:23 -07:00
jarvis-at-skic commented 2026-10-22 22:22:10 -07:00
Author
Owner
Copy link
Copy source

Verified independent review: de2e201c4e

Independent read-only review approved exact tree d46b8b29ef494667a177dd1e41c96556c01cac48 with no blockers. 16 local tests passed before integration and again on the integrated default head. No remote CI workflow is configured. Authenticated readback and runtime comparison verified all 13 tracked package files at c306ea4794f5914d7e4d69a864e2901840895c6e. No scheduler changes or authority/policy widening.

Verified independent review: de2e201c4e85f3af6a9d10c17f5861bb9e4ef44a Independent read-only review approved exact tree `d46b8b29ef494667a177dd1e41c96556c01cac48` with no blockers. 16 local tests passed before integration and again on the integrated default head. No remote CI workflow is configured. Authenticated readback and runtime comparison verified all 13 tracked package files at `c306ea4794f5914d7e4d69a864e2901840895c6e`. No scheduler changes or authority/policy widening.
jarvis-at-skic commented 2026-10-22 22:29:24 -07:00
Author
Owner
Copy link
Copy source

Validation-record clarification: pytest collection — SIMPLIFICATION.md describes an ad-hoc unittest.FunctionTestCase runner, but no such adapter is committed. The final committed-suite verification used pytest, which directly collected all 16 tests, including the three function-style routing tests. The published test result remains accurate. This is a documentation erratum only; no operational skill or gate change.

Validation-record clarification: pytest collection — SIMPLIFICATION.md describes an ad-hoc unittest.FunctionTestCase runner, but no such adapter is committed. The final committed-suite verification used pytest, which directly collected all 16 tests, including the three function-style routing tests. The published test result remains accurate. This is a documentation erratum only; no operational skill or gate change.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
andrius.bertasius (Andrius Bertasius) jarvis-at-skic (Jarvis At Skic) martynas-pazusis (Martynas Pa) oleg-lukasonok (Oleg Lukasonok) root-at-skic (Root At Skic)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: home-v1-skills-code-agent/channel-delivery-skic#13