Skip to content

docs(spiraldb): add pyspiral optional dep and SpiralDB exploration findings#106

Merged
eywalker merged 1 commit intodevfrom
eywalker/plt-1163-connect-devenv-to-spiraldb-and-explore-usage
Mar 25, 2026
Merged

docs(spiraldb): add pyspiral optional dep and SpiralDB exploration findings#106
eywalker merged 1 commit intodevfrom
eywalker/plt-1163-connect-devenv-to-spiraldb-and-explore-usage

Conversation

@kurodo3
Copy link
Copy Markdown
Contributor

@kurodo3 kurodo3 Bot commented Mar 25, 2026

Summary

  • Adds pyspiral>=0.11.0 as the spiraldb optional extra (uv sync --extra spiraldb)
  • Records hands-on pyspiral API exploration in superpowers/specs/2026-03-25-spiraldb-exploration.md

The spec answers all four open design questions blocking the SpiralDBConnector implementation:

  • Constructor scope → dataset-scoped (project_id + dataset)
  • Table naming → plain names within the connector's dataset
  • Write semantics → always upsert-by-key; skip_existing must be approximated client-side via read-then-filter
  • Type fidelity → plain Arrow types except timezone is dropped from vortex.timestamp

Test plan

  • uv sync --extra spiraldb installs pyspiral without errors
  • uv run pyspiral login completes successfully
  • spiral.Spiral().list_projects() returns results

Closes PLT-1163
Related PLT-1074

Adds pyspiral>=0.11.0 as the `spiraldb` optional extra and records
hands-on API exploration results in superpowers/specs/.

Covers devenv setup, table naming, key_schema, type mapping, and
write/upsert semantics — answers the four open design questions
blocking PLT-1074.

Closes PLT-1163

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@eywalker eywalker merged commit 93ddb50 into dev Mar 25, 2026
4 checks passed
@eywalker eywalker deleted the eywalker/plt-1163-connect-devenv-to-spiraldb-and-explore-usage branch March 25, 2026 09:24
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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.

1 participant