Skip to content

feat(storage): add SessionRepository contract #4662

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

Open
MicroGery wants to merge 11 commits into
base: main
Choose a base branch
from
Open

feat(storage): add SessionRepository contract #4662

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
a230e57
feat(storage): add SessionRepository contract
MicroGery Sep 3, 2026
86ad4fb
feat(storage): publish session checkpoint manifests
MicroGery Sep 3, 2026
914b2fc
fix(storage): bind Fork claims to verified sources
MicroGery Sep 3, 2026
9b0b438
fix(storage): retain recoverable Fork sources
MicroGery Sep 4, 2026
6490b3f
feat(storage): materialize retained Fork sources
MicroGery Sep 4, 2026
6fe5ccc
feat(storage): add durable file SessionRepository (#4674)
MicroGery Sep 4, 2026
57d3c0a
fix(ci): compare renderer debt against checked-out merge base
MicroGery Sep 5, 2026
a98c456
Merge remote-tracking branch 'origin/main' into codex/2370-session-re…
MicroGery Sep 5, 2026
8a725e0
fix(ci): share checked-out comparison across diff gates
MicroGery Sep 5, 2026
75d43ee
fix(storage): recover repository writes after process death
MicroGery Sep 5, 2026
958cfb4
fix(storage): align repository identity and recovery semantics
MicroGery Sep 5, 2026
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
45 changes: 34 additions & 11 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -54,16 +54,37 @@ jobs:
fetch-depth: 0
persist-credentials: false

- id: comparison
name: Resolve checked-out comparison
env:
CI_EVENT_NAME: ${{ github.event_name }}
PUSH_BASE_SHA: ${{ github.event.before }}
PR_HEAD_SHA: ${{ github.event.pull_request.head.sha }}
run: |
# Every diff-based gate must compare the same checked-out tree.
# PR event bases can predate the main parent of the actual merge.
BASE_SHA=""
HEAD_SHA="$(git rev-parse --verify HEAD)"
if [[ "$CI_EVENT_NAME" == "pull_request" ]]; then
MERGED_HEAD_SHA="$(git rev-parse --verify 'HEAD^2')"
if [[ -z "$PR_HEAD_SHA" || "$MERGED_HEAD_SHA" != "$PR_HEAD_SHA" ]]; then
echo "CI comparison requires the merge of the requested PR head" >&2
exit 1
fi
BASE_SHA="$(git rev-parse --verify 'HEAD^1')"
elif [[ "$CI_EVENT_NAME" == "push" && ! "$PUSH_BASE_SHA" =~ ^0+$ ]]; then
BASE_SHA="$PUSH_BASE_SHA"
fi
printf 'base=%s\nhead=%s\n' "$BASE_SHA" "$HEAD_SHA" >> "$GITHUB_OUTPUT"

- id: plan
name: Select affected test surfaces
env:
BASE_SHA: ${{ github.event_name == 'push' && github.event.before || github.event.pull_request.base.sha }}
HEAD_SHA: ${{ github.event_name == 'push' && github.sha || github.event.pull_request.head.sha }}
BASE_SHA: ${{ steps.comparison.outputs.base }}
HEAD_SHA: ${{ steps.comparison.outputs.head }}
run: |
# PR checks may predate later main changes, so the planning lane also
# validates the exact merged delta. Dispatches and unavailable history
# fail safe to every surface.
if [[ "${{ github.event_name }}" == "workflow_dispatch" ]] || [[ "$BASE_SHA" =~ ^0+$ ]] || ! git cat-file -e "${BASE_SHA}^{commit}"; then
# Dispatches, initial pushes and unavailable history select every surface.
if [[ -z "$BASE_SHA" ]] || ! git cat-file -e "${BASE_SHA}^{commit}"; then
node scripts/ci-test-plan.mjs --full >> "$GITHUB_OUTPUT"
else
node scripts/ci-test-plan.mjs --base "$BASE_SHA" --head "$HEAD_SHA" >> "$GITHUB_OUTPUT"
Expand Down Expand Up @@ -97,7 +118,9 @@ jobs:
# incompatible protocols under one number (#3313).
- name: Guard the protocol compatibility epoch
if: github.event_name == 'pull_request'
run: node scripts/protocol-epoch-check.mjs --base 'HEAD^1'
env:
BASE_SHA: ${{ steps.comparison.outputs.base }}
run: node scripts/protocol-epoch-check.mjs --base "$BASE_SHA"

- name: Test the epoch guard
run: node --test --test-concurrency=1 scripts/protocol-epoch-check.test.mjs
Expand Down Expand Up @@ -140,10 +163,10 @@ jobs:
- name: Check locale hygiene
if: steps.plan.outputs.code == 'true'
env:
BASE_SHA: ${{ github.event_name == 'push' && github.event.before || github.event.pull_request.base.sha }}
BASE_SHA: ${{ steps.comparison.outputs.base }}
run: |
node --test scripts/check-locale-hygiene.test.mjs
if [[ -n "$BASE_SHA" && ! "$BASE_SHA" =~ ^0+$ ]]; then
if [[ -n "$BASE_SHA" ]]; then
npm run check:locale-hygiene -- --base "$BASE_SHA"
else
npm run check:locale-hygiene
Expand Down Expand Up @@ -214,9 +237,9 @@ jobs:
- name: Check renderer architecture
if: steps.plan.outputs.code == 'true'
env:
BASE_SHA: ${{ github.event_name == 'push' && github.event.before || github.event.pull_request.base.sha }}
BASE_SHA: ${{ steps.comparison.outputs.base }}
run: |
if [[ -n "$BASE_SHA" && ! "$BASE_SHA" =~ ^0+$ ]]; then
if [[ -n "$BASE_SHA" ]]; then
npm run check:renderer-architecture -- --base "$BASE_SHA"
else
npm run check:renderer-architecture
Expand Down
Loading