Skip to content

feat: Add bounded full-catalog import timeout contract - #164

Merged
StatPan merged 1 commit into
mainfrom
issue-163-add-bounded-full-catalog-import-timeout-contract
Jul 22, 2026
Merged

feat: Add bounded full-catalog import timeout contract#164
StatPan merged 1 commit into
mainfrom
issue-163-add-bounded-full-catalog-import-timeout-contract

Conversation

@StatPan

@StatPan StatPan commented Jul 22, 2026

Copy link
Copy Markdown
Owner

Closes #163

@StatPan

StatPan commented Jul 22, 2026

Copy link
Copy Markdown
Owner Author

Check

Self-review packet for ticket #163 and PR #164.

  • PR readiness: blocked
  • Finish ready: false
  • Blockers: checks_pending
  • Diff summary: 3 files changed, +69/-20
    • internal/cli/cli.go: +32/-20
    • internal/cli/cli_test.go: +36/-0
    • internal/datago/importer.go: +1/-0
  • Acceptance mapping candidates:
    • A caller can set a positive full-import deadline explicitly. -> internal/datago/importer.go
    • The deadline applies to the complete import and returns redacted request-failure evidence on expiry. -> internal/datago/importer.go
    • Existing default invocation behavior remains unchanged. -> manual verification required
    • Focused Go tests pass. -> manual verification required
  • Risk areas: Go runtime code changed
  • Full diff: gh pr diff 164 --repo StatPan/datapan-cli

Reviewer notes:

  • Findings: none recorded by this self-review command.
  • Test gaps: verify local and CI evidence before finish.
  • Recommended action: continue normal review/check/finish flow.

Context:

@StatPan
StatPan merged commit e3b2fa4 into main Jul 22, 2026
6 checks passed
@StatPan
StatPan deleted the issue-163-add-bounded-full-catalog-import-timeout-contract branch July 22, 2026 06:15
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.

Add bounded full-catalog import timeout contract

1 participant