mostlyright-data 0.10.5__tar.gz → 0.10.6__tar.gz

{mostlyright_data-0.10.5 → mostlyright_data-0.10.6}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mostlyright-data
-Version: 0.10.5
+Version: 0.10.6
 Summary: Mostly Right CLI for reproducible, reviewed Parquet datasets
 Project-URL: Homepage, https://mostlyright.md/
 Project-URL: Documentation, https://mostlyright.md/docs/guides/cli/

{mostlyright_data-0.10.5 → mostlyright_data-0.10.6}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "mostlyright-data"
-version = "0.10.5"
+version = "0.10.6"
 description = "Mostly Right CLI for reproducible, reviewed Parquet datasets"
 readme = "README.md"
 requires-python = ">=3.13,<3.14"

{mostlyright_data-0.10.5 → mostlyright_data-0.10.6}/skills/mr-data-build/SKILL.md

@@ -50,7 +50,7 @@ routine stages, predict duration or throughput, or expose raw state names. Write
 professional product language and lead with the outcome or decision.
 
 For hosted work, register the Dataset first and open its canonical page immediately in the host's
-in-app browser: `mr-data open /dashboard/datasets/DATASET_ID` mints a single-use address for it, and
+in-app browser: `mr-data open /datasets/DATASET_ID` mints a single-use address for it, and
 when that address cannot be minted, carry on without it rather than blocking.
 Keep that one Dataset tab for research, approval, building, verification, and delivery. Do not
 replace it with a standalone Run, approval, or notebook page unless the user explicitly requests

{mostlyright_data-0.10.5 → mostlyright_data-0.10.6}/src/mostlyright/data_harness/hosted_deploy.py

@@ -2256,7 +2256,7 @@ def _activation_receipt(
         "recipe_proposal_id": state["resources"]["proposal"]["recipe_proposal_id"],
         "state": str(store.path),
         "dashboard_url": (
-            f"{_service_url(cloud_url, 'Cloud URL', allow_loopback_http=False)}/dashboard/datasets"
+            f"{_service_url(cloud_url, 'Cloud URL', allow_loopback_http=False)}/dashboard/team"
         ),
         # The queued Run is not a released version, and this receipt is where somebody learns that
         # for the first time. Its sibling above has always ended by naming the next command; this
@@ -2274,7 +2274,7 @@ def cloud_dashboard_url(cloud_url: str, page: str) -> str:
     """
 
     origin = _service_url(cloud_url, "Cloud URL", allow_loopback_http=False)
-    return f"{origin}/dashboard/workspace/{page}"
+    return f"{origin}/dashboard/{page}"
 
 
 def _bind_cloud_dataset(

{mostlyright_data-0.10.5 → mostlyright_data-0.10.6}/src/mostlyright/data_harness/hosted_run_worker.py

@@ -56,7 +56,6 @@ from dataclasses import dataclass, field, replace
 from pathlib import Path
 from types import FrameType, MappingProxyType
 from typing import Any, ClassVar, NoReturn, Protocol
-from urllib.parse import urlsplit
 from uuid import UUID, uuid5
 
 from mostlyright.data_harness import pipeline, progress_events
@@ -658,25 +657,22 @@ def permitted_artifact_hosts(
     studio_origin: str = "",
     environ: Mapping[str, str] | None = None,
 ) -> frozenset[str]:
-    """The one host this run's artifact bytes -- its own and its predecessor's -- may come from.
+    """The hosts this run's predecessor bytes may come from: the deployment's own policy.
 
-    TWO RULES, AND THE FIRST ONE IS THE DEPLOYMENT'S. ``hosted_worker.signed_storage_hosts`` names
-    the places this fleet's Studio actually signs sessions: the object-storage origin
+    THE RULE IS THE DEPLOYMENT'S. ``hosted_worker.signed_storage_hosts`` names the places this
+    fleet's Studio actually signs sessions: the object-storage origin
     (:data:`~mostlyright.data_harness.hosted_worker.SIGNED_STORAGE_HOST`), the configured Studio
     origin its local backend serves ``/_signed-artifacts/`` from, and whatever
     :data:`~mostlyright.data_harness.hosted_worker.ENV_SIGNED_STORAGE_HOSTS` adds. Every one of
     those is read from the environment or from the coordinator's own configuration, so NOTHING an
     author of a job document can write moves them.
 
-    THE SECOND RULE IS THE DOCUMENT'S OWN CONSISTENCY, and it is second on purpose. The permitted
-    set is still exactly the host ``artifact_session`` names -- the upload session this run will PUT
-    its five sealed artifacts to -- so a predecessor download and this run's own uploads have to
-    agree about where this run's bytes live. That agreement used to be the WHOLE rule, and as the
-    whole rule it was vacuous: ``artifact_session`` reaches no upload (the seal mints its own
-    per-artifact sessions through :class:`StudioArtifactSealer`), so moving it cost an attacker
-    nothing at all and a document naming an internal host in BOTH members passed. Anchoring the
-    agreement inside the deployment's policy is what turns it back into a constraint: the sibling
-    member can only be moved to somewhere Studio genuinely signs.
+    The document's own ``artifact_session`` has to sit inside that policy too, and it is checked
+    here so a document naming somewhere Studio does not sign is refused before a byte moves. It is
+    NOT the anchor the predecessor download is narrowed to: that member is the
+    ``artifacts:reserve`` route on Studio's own origin, while a predecessor's sealed bundle is
+    signed on the object store. The two are different hosts by design, and an earlier rule that
+    required them to agree refused every refresh a hosted deployment dispatched.
 
     This is defence in depth and it is not the authentication: the predecessor's bytes are still
     authenticated end to end by ``candidate_digest``, checked by ``_decode_candidate_envelope``
@@ -697,10 +693,7 @@ def permitted_artifact_hosts(
         # NEVER THE URL, for the reason `_download_signed_bytes` records: the address carries a
         # credential in its query string and `signed_storage_refusal` is written never to echo it.
         raise _host_refusal(f"this run's artifact session is not one Studio signs: {reason}")
-    host = urlsplit(session.signed_url).hostname
-    if not host:  # pragma: no cover - `signed_storage_refusal` already required a host.
-        raise _host_refusal("this run's artifact session names no host")
-    return frozenset({host.lower()})
+    return policy
 
 
 def _download_signed_bytes(
@@ -2833,12 +2826,11 @@ def _resolved_sealer(
         transport=stack.enter_context(_closing(_new_transfer_client())),
         transfer=transfer,
         base_url=base_url,
-        # THE DEPLOYMENT'S POLICY, NOT THE DOCUMENT'S OWN HOST. `permitted_artifact_hosts` narrows
-        # this set to the one host the job's `artifact_session` names, which is right for the
-        # predecessor download -- both are bytes of the same run -- and wrong here: that member is
+        # THE DEPLOYMENT'S POLICY, NOT THE DOCUMENT'S OWN HOST. The job's `artifact_session` is
         # the `artifacts:reserve` route on Studio's own origin, while the session it answers with
         # is signed on the object store. The two are DIFFERENT hosts by design, and the rule the
-        # answer has to satisfy is the one `signed_storage_hosts` reads out of deployment facts.
+        # answer has to satisfy is the one `signed_storage_hosts` reads out of deployment facts --
+        # the same rule `permitted_artifact_hosts` applies to the predecessor download.
         permitted_hosts=signed_storage_hosts(studio_origin=base_url, environ=os.environ),
     )
 

{mostlyright_data-0.10.5 → mostlyright_data-0.10.6}/src/mostlyright/data_harness/thin/approvals.py

@@ -60,7 +60,7 @@ from mostlyright.data_harness.thin.runs import StudioApiClient, new_idempotency_
 from mostlyright.data_harness.thin.session import (
     StudioSession,
     open_studio_session,
-    run_dashboard_url,
+    run_record_dashboard_url,
 )
 from mostlyright.data_harness.thin.transport import ThinLaneError
 from mostlyright.data_harness.ux.credentials import resolve_cloud_credentials
@@ -785,7 +785,7 @@ def _confirm_table_recipe(
         payload["build"] = {
             "state": "run_already_queued",
             "run_id": run_id,
-            "dashboard_url": run_dashboard_url(selected.session.cloud_url, run_id),
+            "dashboard_url": run_record_dashboard_url(selected.session.cloud_url, run, run_id),
             "watch_command": f"mr-data watch {run_id} --hosted",
             "note": (
                 "The confirmation started the first hosted Build in the same act; there is "
@@ -823,7 +823,7 @@ def _what_follows_an_approved_recipe(
         return {
             "state": "run_already_queued",
             "run_id": run_id,
-            "dashboard_url": run_dashboard_url(client.session.cloud_url, run_id),
+            "dashboard_url": run_record_dashboard_url(client.session.cloud_url, run, run_id),
             "watch_command": f"mr-data watch {run_id} --hosted",
             "note": (
                 "The confirmation started the first hosted Build in the same act; there is "

{mostlyright_data-0.10.5 → mostlyright_data-0.10.6}/src/mostlyright/data_harness/thin/commands.py

@@ -28,7 +28,7 @@ from mostlyright.data_harness.thin.runs import RESOURCE_CLASSES, StudioRunClient
 from mostlyright.data_harness.thin.session import (
     StudioSession,
     open_studio_session,
-    run_dashboard_url,
+    run_record_dashboard_url,
 )
 from mostlyright.data_harness.thin.stream import TERMINAL_RUN_STATUSES, RunEventStream
 from mostlyright.data_harness.thin.transport import ThinLaneError
@@ -115,7 +115,7 @@ def submit(args: argparse.Namespace, *, client: StudioRunClient | None = None) -
         "schema_version": SUBMIT_SCHEMA,
         "status": "run_queued",
         "lane": "hosted",
-        "dashboard_url": run_dashboard_url(selected.session.cloud_url, run_id),
+        "dashboard_url": run_record_dashboard_url(selected.session.cloud_url, run, run_id),
         "watch_command": f"mr-data watch {run_id} --hosted",
         "run": run_receipt(run),
         "studio": selected.session.to_receipt(),
@@ -158,7 +158,7 @@ def _already_started(
         "status": "run_already_queued",
         "lane": "hosted",
         "approval_request_id": approval_request_id,
-        "dashboard_url": run_dashboard_url(client.session.cloud_url, run_id),
+        "dashboard_url": run_record_dashboard_url(client.session.cloud_url, run, run_id),
         "watch_command": f"mr-data watch {run_id} --hosted",
         "run": run_receipt(run),
         "note": (
@@ -189,8 +189,11 @@ def watch(args: argparse.Namespace, *, client: StudioRunClient | None = None) ->
             print(line, flush=True)
 
     if announce:
+        # The run is read once before the timeline so the address printed first is the run's
+        # Dataset page; the read below is the terminal record, made after the timeline ends.
         print(f"watching {run_id}", flush=True)
-        print(run_dashboard_url(selected.session.cloud_url, run_id), flush=True)
+        queued = selected.get_run(run_id)
+        print(run_record_dashboard_url(selected.session.cloud_url, queued, run_id), flush=True)
     progress = stream.follow(
         from_seq=int(getattr(args, "from_seq", 0) or 0),
         last_event_id=getattr(args, "last_event_id", None),
@@ -201,7 +204,7 @@ def watch(args: argparse.Namespace, *, client: StudioRunClient | None = None) ->
         "schema_version": WATCH_SCHEMA,
         "status": "run_terminal" if run.get("status") in TERMINAL_RUN_STATUSES else "watch_ended",
         "lane": "hosted",
-        "dashboard_url": run_dashboard_url(selected.session.cloud_url, run_id),
+        "dashboard_url": run_record_dashboard_url(selected.session.cloud_url, run, run_id),
         "run": run_receipt(run),
         "stream": progress.to_receipt(),
     }
@@ -224,7 +227,7 @@ def status(args: argparse.Namespace, *, client: StudioRunClient | None = None) -
         "schema_version": STATUS_SCHEMA,
         "status": "run_reported",
         "lane": "hosted",
-        "dashboard_url": run_dashboard_url(selected.session.cloud_url, run_id),
+        "dashboard_url": run_record_dashboard_url(selected.session.cloud_url, run, run_id),
         "run": run_receipt(run),
     }
     if not getattr(args, "receipts", False):
@@ -292,6 +295,9 @@ def download(args: argparse.Namespace, *, client: StudioRunClient | None = None)
             f"this run published no artifact of kind {', '.join(wanted)}"
             + (f"; it published {', '.join(available)}" if available else ""),
         )
+    # The evidence names artifacts and not the Dataset; the run record is what names the Dataset
+    # page the receipt points at.
+    run = selected.get_run(run_id)
     downloaded = []
     for artifact in selectedartifact_records:
         artifact_id = artifact.get("artifact_id")
@@ -304,7 +310,7 @@ def download(args: argparse.Namespace, *, client: StudioRunClient | None = None)
         "schema_version": DOWNLOAD_SCHEMA,
         "status": "artifacts_downloaded",
         "lane": "hosted",
-        "dashboard_url": run_dashboard_url(selected.session.cloud_url, run_id),
+        "dashboard_url": run_record_dashboard_url(selected.session.cloud_url, run, run_id),
         "run_id": run_id,
         "output": str(output),
         "artifacts": downloaded,

{mostlyright_data-0.10.5 → mostlyright_data-0.10.6}/src/mostlyright/data_harness/thin/parity.py

@@ -46,7 +46,8 @@ from mostlyright.data_harness.thin.runs import StudioRunClient
 from mostlyright.data_harness.thin.session import (
     StudioSession,
     open_studio_session,
-    run_dashboard_url,
+    run_record_dashboard_url,
+    team_dashboard_url,
 )
 from mostlyright.data_harness.thin.stream import TERMINAL_RUN_STATUSES
 from mostlyright.data_harness.thin.transport import ThinLaneError
@@ -252,13 +253,24 @@ def _numbered(prefix: str, items: Sequence[Any]) -> dict[str, Any]:
     return {f"{prefix} {index:0{width}d}": item for index, item in enumerate(items, start=1)}
 
 
-def _hosted_head(session: StudioSession, run_id: str) -> dict[str, Any]:
-    """The three facts every answer in this module opens with."""
+def _hosted_head(
+    session: StudioSession, run_id: str, run: Mapping[str, Any] | None = None
+) -> dict[str, Any]:
+    """The three facts every answer in this module opens with.
+
+    The address is the run's Dataset page when the command read the run record, and the team's
+    Dataset listing when it did not: the notebook commands read the rendered notebook alone and
+    hold no Dataset identifier to name a page with.
+    """
 
     return {
         "lane": "hosted",
         "run_id": run_id,
-        "dashboard_url": run_dashboard_url(session.cloud_url, run_id),
+        "dashboard_url": (
+            run_record_dashboard_url(session.cloud_url, run, run_id)
+            if run is not None
+            else team_dashboard_url(session.cloud_url)
+        ),
     }
 
 
@@ -305,7 +317,7 @@ def show(args: argparse.Namespace, *, client: StudioRunClient | None = None) ->
     payload: dict[str, Any] = {
         "schema_version": SHOW_SCHEMA,
         "status": "build_shown",
-        **_hosted_head(selected.session, run_id),
+        **_hosted_head(selected.session, run_id, run),
         "run": run_receipt(run),
         **_evidence_receipt(evidence),
     }
@@ -332,7 +344,7 @@ def show(args: argparse.Namespace, *, client: StudioRunClient | None = None) ->
         return {
             "schema_version": SHOW_SCHEMA,
             "status": "build_shown",
-            **_hosted_head(selected.session, run_id),
+            **_hosted_head(selected.session, run_id, run),
             "card": payload["card"],
         }
     payload["receipts_unavailable"] = unavailable
@@ -414,7 +426,7 @@ def inspect(args: argparse.Namespace, *, client: StudioRunClient | None = None)
     return {
         "schema_version": INSPECT_SCHEMA,
         "status": "candidate_inspected",
-        **_hosted_head(selected.session, run_id),
+        **_hosted_head(selected.session, run_id, run),
         "run": run_receipt(run),
         **_evidence_receipt(evidence),
         "execution_summary": summary,
@@ -453,7 +465,7 @@ def verify(args: argparse.Namespace, *, client: StudioRunClient | None = None) -
     return {
         "schema_version": VERIFY_SCHEMA,
         "status": "fixes_required" if blocking else "candidate_verified",
-        **_hosted_head(selected.session, run_id),
+        **_hosted_head(selected.session, run_id, run),
         "run": run_receipt(run),
         **_evidence_receipt(evidence),
         "checked_by": "the independent hosted check, read back; the bytes were not replayed here",
@@ -949,7 +961,7 @@ def peek(args: argparse.Namespace, *, client: StudioRunClient | None = None) ->
     payload: dict[str, Any] = {
         "schema_version": PEEK_SCHEMA,
         "status": "peeked",
-        **_hosted_head(selected.session, run_id),
+        **_hosted_head(selected.session, run_id, run),
         "source": f"{table_id}/{version_id}",
         "origin": "hosted",
         "table_id": table_id,
@@ -1024,7 +1036,7 @@ def deployment_status(
     return {
         "schema_version": DEPLOYMENT_STATUS_SCHEMA,
         "status": "run_status_reported",
-        **_hosted_head(selected.session, run_id),
+        **_hosted_head(selected.session, run_id, run),
         "run_status": state,
         "terminal": state in TERMINAL_RUN_STATUSES,
         "workspace_id": str(selected.session.workspace_id),

{mostlyright_data-0.10.5 → mostlyright_data-0.10.6}/src/mostlyright/data_harness/thin/propose.py

@@ -93,7 +93,7 @@ from mostlyright.data_harness.thin.session import (
     approval_dashboard_url,
     dataset_dashboard_url,
     open_studio_session,
-    run_dashboard_url,
+    run_record_dashboard_url,
 )
 from mostlyright.data_harness.thin.transport import ThinLaneError
 from mostlyright.data_harness.ux.credentials import resolve_cloud_credentials
@@ -2776,7 +2776,12 @@ def _proposal_receipt(
         )
     payload["status"] = "recipe_build_queued"
     payload["run_id"] = run_id
-    payload["run_dashboard_url"] = run_dashboard_url(client.session.cloud_url, run_id)
+    # The journal already names the Dataset this bundle registered; the started run's own record
+    # names the same one, and either is enough to put the run on its Dataset's page.
+    dataset_id = payload["ids"].get("dataset_id") or run.get("dataset_id")
+    payload["run_dashboard_url"] = run_record_dashboard_url(
+        client.session.cloud_url, {"dataset_id": dataset_id}, run_id
+    )
     payload["watch_command"] = f"mr-data watch {run_id} --hosted"
     payload["note"] = (
         "The confirmation started the first hosted Build in the same act; there is nothing left "

{mostlyright_data-0.10.5 → mostlyright_data-0.10.6}/src/mostlyright/data_harness/thin/research.py

@@ -676,7 +676,7 @@ def _dashboard_urls(cloud_url: str, record: Mapping[str, Any], run_id: str) -> d
 
     dataset_id = _dataset_identifier(record)
     return {
-        "dashboard_url": run_dashboard_url(cloud_url, run_id),
+        "dashboard_url": run_dashboard_url(cloud_url, dataset_id, run_id),
         "dataset_dashboard_url": dataset_dashboard_url(cloud_url, dataset_id),
         "research_dashboard_url": dataset_research_dashboard_url(cloud_url, dataset_id, run_id),
     }

{mostlyright_data-0.10.5 → mostlyright_data-0.10.6}/src/mostlyright/data_harness/thin/session.py

@@ -38,22 +38,25 @@ from mostlyright.data_harness.ux.credentials import ResolvedCloudCredentials
 #: Cloud's token-exchange route. Held identical to ``hosted_deploy.CLOUD_TOKEN_PATH`` by a test.
 CLOUD_TOKEN_PATH = "/api/cli/studio-token"
 
-#: The Cloud page a person opens about one run. ADR 0021 makes the live run view the place a
-#: hosted submission points at, and ``mostlyright-cloud`` serves it at this route
-#: (``dashboard/app/dashboard/runs/[runId]/page.tsx``). The origin is never hardcoded beyond the
-#: configured Cloud URL, which ``MOSTLYRIGHT_CLOUD_URL`` overrides and the stored credential
-#: carries.
-CLOUD_RUN_DASHBOARD_PATH = "/dashboard/runs/{run_id}"
+#: The one Cloud address a Dataset has. ``mostlyright-cloud`` serves it at this route with
+#: section suffixes, and a run has no page of its own: its evidence lives on this page's Harness
+#: card, so a receipt names its run with :func:`run_dashboard_url` as a query member here rather
+#: than as a path. The origin is never hardcoded beyond the configured Cloud URL, which
+#: ``MOSTLYRIGHT_CLOUD_URL`` overrides and the stored credential carries.
+CLOUD_DATASET_DASHBOARD_PATH = "/datasets/{dataset_id}"
 
-#: The durable Cloud home of one Dataset. Research starts here before a session exists, then the
-#: same page follows the exact session run through :func:`dataset_research_dashboard_url`.
-CLOUD_DATASET_DASHBOARD_PATH = "/dashboard/datasets/{dataset_id}"
+#: The query member that selects one run on the Dataset page.
+CLOUD_DATASET_RUN_QUERY_KEY = "run"
+
+#: The Cloud listing of this team's Datasets, printed by the receipts that hold no Dataset
+#: identifier and therefore cannot name one Dataset page.
+CLOUD_TEAM_DASHBOARD_PATH = "/dashboard/team"
 
 #: The Cloud page a person opens to settle one approval. ``mostlyright-cloud`` serves it at
-#: ``dashboard/app/dashboard/approvals/[approvalRequestId]/page.tsx`` -- the sibling of the run
-#: page above, and the address ``mr-data propose`` prints for the human who must grant the Table
-#: recipe approval it opened. It is where a person goes; the actual step-up or editor confirmation
-#: is what ``mr-data recipe-approve`` records once they have.
+#: ``dashboard/app/dashboard/approvals/[approvalRequestId]/page.tsx`` -- the sibling of the
+#: Dataset page above, and the address ``mr-data propose`` prints for the human who must grant the
+#: Table recipe approval it opened. It is where a person goes; the actual step-up or editor
+#: confirmation is what ``mr-data recipe-approve`` records once they have.
 CLOUD_APPROVAL_DASHBOARD_PATH = "/dashboard/approvals/{approval_request_id}"
 
 MAX_TOKEN_RESPONSE_BYTES = 32 * 1024
@@ -95,37 +98,60 @@ class StudioSession:
         }
 
 
-def run_dashboard_url(cloud_url: str, run_id: str) -> str:
-    """The Cloud live-run page for one run, from one validated origin."""
+def team_dashboard_url(cloud_url: str) -> str:
+    """The Cloud listing of this team's Datasets, from one validated origin."""
 
     origin = service_origin(cloud_url, "Cloud URL", allow_loopback_http=False)
-    return origin + CLOUD_RUN_DASHBOARD_PATH.format(run_id=run_id)
+    return origin + CLOUD_TEAM_DASHBOARD_PATH
 
 
 def dataset_dashboard_url(cloud_url: str, dataset_id: str) -> str:
-    """The Cloud detail page for one Dataset, with its identifier held to one path segment."""
+    """The Cloud page for one Dataset, with its identifier held to one path segment."""
 
     origin = service_origin(cloud_url, "Cloud URL", allow_loopback_http=False)
     encoded_dataset_id = quote(str(dataset_id), safe="")
     return origin + CLOUD_DATASET_DASHBOARD_PATH.format(dataset_id=encoded_dataset_id)
 
 
-def dataset_research_dashboard_url(cloud_url: str, dataset_id: str, run_id: str) -> str:
-    """The exact research run embedded in one Dataset detail page.
+def run_dashboard_url(cloud_url: str, dataset_id: str, run_id: str) -> str:
+    """One run, named on its Dataset's page.
 
     Path and query coordinates are encoded independently so neither can introduce a sibling path,
-    another query member, or a fragment. The fragment is the fixed UI section identifier.
+    another query member, or a fragment.
     """
 
     base = dataset_dashboard_url(cloud_url, dataset_id)
-    query = urlencode({"run": str(run_id)}, quote_via=quote, safe="")
-    return f"{base}?{query}#research"
+    query = urlencode({CLOUD_DATASET_RUN_QUERY_KEY: str(run_id)}, quote_via=quote, safe="")
+    return f"{base}?{query}"
+
+
+def run_record_dashboard_url(cloud_url: str, run: Mapping[str, Any], run_id: str) -> str:
+    """The address a run record answers with: its Dataset's page, or the team listing.
+
+    A run record names its Dataset, and that is the page a person opens about the run. A record
+    that names none -- a v3 projection that carried no ``dataset_id`` -- gets the team's Dataset
+    listing rather than a run page this product no longer has.
+    """
+
+    dataset_id = run.get("dataset_id")
+    if isinstance(dataset_id, str) and dataset_id:
+        return run_dashboard_url(cloud_url, dataset_id, run_id)
+    return team_dashboard_url(cloud_url)
+
+
+def dataset_research_dashboard_url(cloud_url: str, dataset_id: str, run_id: str) -> str:
+    """The exact research run embedded in one Dataset page.
+
+    The fragment is the fixed UI section identifier.
+    """
+
+    return f"{run_dashboard_url(cloud_url, dataset_id, run_id)}#research"
 
 
 def approval_dashboard_url(cloud_url: str, approval_request_id: str) -> str:
     """The Cloud approvals page for one approval request, from the same validated origin.
 
-    The sibling of :func:`run_dashboard_url`: ``mr-data propose`` opens a Table recipe approval
+    The sibling of :func:`dataset_dashboard_url`: ``mr-data propose`` opens a Table recipe approval
     and hands a person this address to settle it, the way ``mr-data approve`` prints the human
     action Studio names. Both go through :func:`service_origin`, so the origin is checked once and
     the two surfaces cannot drift into two spellings of one Cloud URL.
@@ -265,7 +291,8 @@ def open_studio_session(
 __all__ = [
     "CLOUD_APPROVAL_DASHBOARD_PATH",
     "CLOUD_DATASET_DASHBOARD_PATH",
-    "CLOUD_RUN_DASHBOARD_PATH",
+    "CLOUD_DATASET_RUN_QUERY_KEY",
+    "CLOUD_TEAM_DASHBOARD_PATH",
     "CLOUD_TOKEN_PATH",
     "MAX_JSON_RESPONSE_BYTES",
     "MAX_TOKEN_RESPONSE_BYTES",
@@ -278,4 +305,6 @@ __all__ = [
     "open_studio_session",
     "require_device_key",
     "run_dashboard_url",
+    "run_record_dashboard_url",
+    "team_dashboard_url",
 ]

{mostlyright_data-0.10.5 → mostlyright_data-0.10.6}/src/mostlyright/data_harness/thin/v4_artifacts.py

@@ -56,7 +56,8 @@ from mostlyright.data_harness.thin.download import download_signed_artifact
 from mostlyright.data_harness.thin.session import (
     StudioSession,
     open_studio_session,
-    run_dashboard_url,
+    run_record_dashboard_url,
+    team_dashboard_url,
 )
 from mostlyright.data_harness.thin.transport import ThinLaneError
 from mostlyright.data_harness.thin.v4 import (
@@ -384,13 +385,24 @@ def _document(
     return parsed
 
 
-def _head(session: StudioSession, run_id: str) -> dict[str, Any]:
-    """The three facts every answer in this module opens with."""
+def _head(
+    session: StudioSession, run_id: str, run: Mapping[str, Any] | None = None
+) -> dict[str, Any]:
+    """The three facts every answer in this module opens with.
+
+    The address is the run's Dataset page when the command read the run record, and the team's
+    Dataset listing when it did not: ``download``, ``receipt`` and ``peek`` read nothing but
+    artifacts, so they hold no Dataset identifier to name a page with.
+    """
 
     return {
         "lane": "hosted",
         "run_id": run_id,
-        "dashboard_url": run_dashboard_url(session.cloud_url, run_id),
+        "dashboard_url": (
+            run_record_dashboard_url(session.cloud_url, run, run_id)
+            if run is not None
+            else team_dashboard_url(session.cloud_url)
+        ),
     }
 
 
@@ -503,7 +515,7 @@ def checks(
     payload: dict[str, Any] = {
         "schema_version": CHECKS_SCHEMA,
         "status": "checks_reported",
-        **_head(selected.session, run_id),
+        **_head(selected.session, run_id, run),
         "run_status": state,
     }
     if state == "failed":

{mostlyright_data-0.10.5 → mostlyright_data-0.10.6}/src/mostlyright/data_harness/thin/v4_query.py

@@ -48,7 +48,7 @@ from mostlyright.data_harness.thin import numbered as _numbered
 from mostlyright.data_harness.thin.session import (
     StudioSession,
     open_studio_session,
-    run_dashboard_url,
+    team_dashboard_url,
 )
 from mostlyright.data_harness.thin.transport import ThinLaneError
 from mostlyright.data_harness.thin.v4 import (
@@ -432,7 +432,7 @@ def query(
         "lane": "hosted",
         "run_id": run_id,
         "query_id": query_id,
-        "dashboard_url": run_dashboard_url(selected.session.cloud_url, run_id),
+        "dashboard_url": team_dashboard_url(selected.session.cloud_url),
         "replayed": replayed,
         "polls": read,
     }

{mostlyright_data-0.10.5 → mostlyright_data-0.10.6}/src/mostlyright/data_harness/thin/v4_runs.py

@@ -32,7 +32,7 @@ from mostlyright.data_harness.thin import numbered as _numbered
 from mostlyright.data_harness.thin.session import (
     StudioSession,
     open_studio_session,
-    run_dashboard_url,
+    run_record_dashboard_url,
 )
 from mostlyright.data_harness.thin.transport import ThinLaneError
 from mostlyright.data_harness.thin.v4 import (
@@ -481,7 +481,7 @@ def _started(run: Mapping[str, Any], session: StudioSession, *, status: str) ->
         "schema_version": START_SCHEMA,
         "status": status,
         "lane": "hosted",
-        "dashboard_url": run_dashboard_url(session.cloud_url, run_id),
+        "dashboard_url": run_record_dashboard_url(session.cloud_url, run, run_id),
         "watch_command": f"mr-data watch {run_id}",
         "run": _run_receipt(run),
     }