Remove Stale Checkpoint Summary Repo Parameter

b0eed5c→main·

pfleidi·3w ago·2 files·+2 added/-4 removed

Drop the repository argument left behind after policy enforcement moved out of summary generation.

This keeps the summary helper signature aligned with the data it actually uses.

Sessions

16e73f62b845View transcript

[?
Enforce Checkpoint Policies in CLICodex·GPT-5.5·1 step](/content/gh/entireio/cli/session/019f05ad-eea0-7202-a508-ec34d069a2d2#timeline-16e73f62b845/index.html)

Changes

2

• cmd/entire/cli

• Mexplain.go+2/-2

• Mexplain_test.go-2

690 unmodified lines

691
692
693
694
694
695
696
697
205 unmodified lines

903
904
905
906
906
907
908
909

690 unmodified lines

if openErr != nil {
        return fmt.Errorf("open checkpoint store: %w", openErr)
    }
    if err := generateCheckpointSummary(ctx, w, errW, lookup.repo, writeStores.Persistent, fullCheckpointID, summary, content, force, summaryTimeoutSeconds); err != nil {
    if err := generateCheckpointSummary(ctx, w, errW, writeStores.Persistent, fullCheckpointID, summary, content, force, summaryTimeoutSeconds); err != nil {
        return err
    }
        // Reload to get the updated summary.
205 unmodified lines

// summaryTimeoutSeconds is the per-invocation --summary-timeout-seconds flag
// value (0 = unset). Effective precedence for the deadline: flag > settings >
// package default. See resolveSummaryTimeout for the resolution.
func generateCheckpointSummary(ctx context.Context, w, errW io.Writer, repo *git.Repository, store checkpoint.Writer, checkpointID id.CheckpointID, cpSummary *checkpoint.CheckpointSummary, content *checkpoint.SessionContent, force bool, summaryTimeoutSeconds int) error {
func generateCheckpointSummary(ctx context.Context, w, errW io.Writer, store checkpoint.Writer, checkpointID id.CheckpointID, cpSummary *checkpoint.CheckpointSummary, content *checkpoint.SessionContent, force bool, summaryTimeoutSeconds int) error {
    // Check if summary already exists
    if content.Metadata.Summary != nil && !force {
        return renderExplainFailure(errW, "Summary already exists", []explainRow{

Mcmd/entire/cli/explain.go+2/-2

1108 unmodified lines

1109
1110
1111
1112
1112
1113
1114
21 unmodified lines

1136
1137
1138
1140
1139
1140
1141

1108 unmodified lines

fixture.ctx,
    &stdout,
    &stderr,
    fixture.repo,
    fixture.store,
    fixture.cpID,
    fixture.cpSummary,
21 unmodified lines

fixture.ctx,
    &stdout,
    &stderr,
    fixture.repo,
    fixture.store,
    fixture.cpID,
    fixture.cpSummary,

Mcmd/entire/cli/explain_test.go-2