Skip to content

rust(feat): return url after import#656

Open
Brandon-Shippy wants to merge 13 commits into
mainfrom
rust/return-url-after-import
Open

rust(feat): return url after import#656
Brandon-Shippy wants to merge 13 commits into
mainfrom
rust/return-url-after-import

Conversation

@Brandon-Shippy

@Brandon-Shippy Brandon-Shippy commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Description

This PR adds functionality to build deep explore urls directly linking to the asset/run that the user just imported.
Adds a section in the CLI config for users to input their web-uri to properly resolve links back to sift. Adds extra flow to config creation to handle optional web_uri. If users omit web_uri the CLI will not produce links and the agent will also surface no link to the user. Also adds a helper to help with url building, and percent encoding.

If a URL is provided in the config, that overrides and we use that. If not, we check their api and see if they are on prod or gov, if so we resolve the link to be on prod or gov. If not, we check and see if they are on local and if so we resolve it to local. Otherwise, we just provide no link and surface to the user why.

Note: If a --run-id is provided we will build the link using that. We follow the same idiomatic system that the CLI already follow: --run-id takes precedence over --run

Bundled with this PR is also:

  • Skills for Claude to surface the link back to the user
  • Fixes for service/explore_url that was assuming that urls are deterministic, so we can have the user include their web-url to adjust for on prem deployments and other web urls user can have
  • Skill update so that the agent does not auto-swap profiles on failure and instead surfaces it to the user and asks what is best to do
  • Adds explore_url param to wait for completion so that it can resolve the link on both --wait imports and non --wait imports

Validation

  • Manual imports and checking if links resolve
  • Mass imports via MCP of all import types and ensuring it gets returned properly

@Brandon-Shippy Brandon-Shippy marked this pull request as ready for review June 25, 2026 18:40
Comment thread rust/crates/sift_cli/assets/skills/agents-md/AGENTS.md Outdated
Comment thread rust/crates/sift_cli/src/cmd/import/parquet/flat_dataset.rs Outdated
Comment thread rust/crates/sift_cli/src/cmd/import/parquet/flat_dataset.rs
@Brandon-Shippy Brandon-Shippy force-pushed the rust/return-url-after-import branch from 0246214 to 00e1851 Compare June 26, 2026 04:46
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.

3 participants