[karhu]

The Stack

This stack covers refactoring the internal logic of how releases are resolved during read, download & build, in particular also in the context of parent releases / parent datasets. On top of that it refactors the public TimeNet APIs and CLI interfaces around these actions, to streamline the behavior across them.

This PR

TimeNet.get() is replaced by get_manifest() and get_card(). timenet info uses get_manifest().

Details

TimeNet.get() asked one registry of the list: the first registry that is not the store. So its manifest can belong to another release than the release that open() reads. It also knows nothing of a release that only an installed connector provides.

• get_manifest() returns the manifest of the release that open() reads. It resolves the release through the whole list and reads only the manifest. If a registry lists a release but does not serve its files, it returns the listed manifest, so the access pages stay visible.
• A build creates the manifest of a release. For a release that only an installed connector provides, get_manifest() raises the new TimeNetNotBuiltError, which names get_card() and timenet.build(). download() raises the same error for such a release.
• get_card() returns the card of the release, from its manifest or from the installed connector.
• The two methods take org/name@version like open() and add its hints. They share the hints with open() in one private context manager.
• For a release that no source provides, the two methods raise the error of raise_not_found() in the read coordinator, as open() and download() do. So a missing release gives the same error in each of them.
• timenet info shows the card and the build command for a release that only an installed connector provides. The build command repeats the --local-registry and --remote-registry that info got.
• Size: about 560 changed lines.

Design decisions

• get_manifest() resolves through the whole list, because a manifest of another release than the one that open() reads is misleading.
• A release that only a connector provides raises TimeNetNotBuiltError and not a manifest with missing fields, because the manifest does not exist before the build. The error names the two ways out.
• The shared error function keeps the read plan and the client in step, so a missing release gives one error in both.

Breaking changes

• TimeNet.get() is gone. get_manifest() returns the manifest of the release that open() reads and raises TimeNetNotBuiltError for a release that only a connector provides. get_card() returns its card.

Stack

• docs: update the docs for the build coordinator #683
• feat(hints)!: give every error one hint for the SDK and one for the CLI #698
• feat(cache)!: record where each release in the local registry comes from #674
• feat(client)!: list and search every source #673
• 👉 feat(client)!: replace get() with get_manifest() and get_card() #672
• feat(build)!: build into the local registry, publish with --publish-to #671
• feat!: add timenet.build() and publish() #670
• feat(cli)!: take --local-registry, --remote-registry and --auto-build #669
• feat(config)!: make the default list of sources from three variables #668
• test: isolate every test from the real home and guard it #667
• feat(client)!: download releases into the local registry #662
• feat(client)!: take an ordered list of sources #661
• feat(client)!: read releases through the coordinator #660
• feat(coordinator)!: plan and run reads through the list of sources #659
• feat(coordinator)!: plan parents through an ordered list of sources #658
• feat(sources)!: resolve a release through an ordered list of sources #657
• feat(build)!: run every build through the coordinator #655
• feat(build)!: move the build CLI and build() onto the coordinator #654
• feat(build)!: build one prepared release in the built-in backend #653
• feat(coordinator)!: plan and run builds through build_release #652
• refactor(coordinator)!: name the plan step type Step #651
• feat(registry)!: copy a release between registries #649
• feat(registry)!: publish a local release with put_release #648
• refactor(build)!: let the connector choose the values backend #646

Changes later in the stack:

• docs: update the docs for the build coordinator #683 holds the doc changes of this PR.
• list() and search() still ask one registry through _catalog(). The next PR replaces them.

Merge message

feat(client)!: replace get() with get_manifest() and get_card()

get_manifest() returns the manifest of the release that open() reads,
because it resolves through the whole list of sources. get() asked
only one registry, so its manifest could belong to another release.

- For a release that only an installed connector provides,
  get_manifest() and download() raise the new TimeNetNotBuiltError,
  which names get_card() and timenet.build().
- For a release that a registry lists but does not serve,
  get_manifest() returns the listed manifest, so the access pages stay
  visible.
- get_card() returns the card of the release, from its manifest or
  from the connector.
- timenet info shows the card and the build command for a release that
  is not built yet.

BREAKING CHANGE: TimeNet.get() is gone. Use get_manifest() or
get_card().

[TimoStoff]

lgtm!

[claude]

Code review completed

Nothing new to post: everything this review found is already covered by existing comments on this pull request or didn't merit a separate one.

[claude]

Code review completed

Nothing new to post: everything this review found is already covered by existing comments on this pull request or didn't merit a separate one.

Still open from earlier reviews (1):

• 🔴 packages/timenet/src/timenet/client.py:528 — Users with a connector installed lose the listed manifest of gated releases. get_manifest() and timenet info raise Time…

If you have decided not to act on one of these findings, resolve its thread (a reply alone leaves it open) and the next review stops counting it. To review this commit again now, use Re-run on its "Claude Code Review" check.