Skip to content

Redirect sandbox-harbor to harbor-integrations#sandboxes#4635

Open
Nicholas Bohm (Nickbohm555) wants to merge 1 commit into
mainfrom
nickbohm/sandbox-redirect-url
Open

Redirect sandbox-harbor to harbor-integrations#sandboxes#4635
Nicholas Bohm (Nickbohm555) wants to merge 1 commit into
mainfrom
nickbohm/sandbox-redirect-url

Conversation

@Nickbohm555

Copy link
Copy Markdown
Contributor

Why

The standalone "Run evaluations with Harbor" page (/langsmith/sandbox-harbor) in the Sandboxes tab duplicated content that now lives in the canonical Harbor integrations evals doc (/langsmith/harbor-integrations, which has a dedicated ## Sandboxes section). This consolidates the two so there is a single source of truth, and preserves existing links/bookmarks via a redirect.

Changes

  • Redirect /langsmith/sandbox-harbor/langsmith/harbor-integrations#sandboxes (src/docs.json redirects).
  • Sidebar: the Sandboxes-tab entry now points at the evals doc (langsmith/harbor-integrations) instead of the removed page.
  • Removed src/langsmith/sandbox-harbor.mdx (required for the redirect, per check_removed_pages_redirects).
  • Updated the "Harbor" card in sandboxes.mdx to the new anchored URL.
  • Removed the now self-referential "Run evaluations with Harbor" item from the harbor-integrations.mdx "See also" list.

Review notes

  • The #sandboxes anchor resolves: harbor-integrations.mdx has a ## Sandboxes heading.
  • harbor-integrations now appears in both the Evaluation tab and the Sandboxes tab — intentional, so the Sandboxes sidebar links to the evals doc.

Verification

  • check_removed_pages_redirects.py (pre-commit): passed.
  • make lint_prose on changed files: 0 errors.
  • Local dev server (Node 22): /langsmith/sandbox-harbor returns 307/langsmith/harbor-integrations#sandboxes; target returns 200.
  • make broken-links not run locally (mint requires Node ≤24; this machine is on Node 26) — relying on CI.

AI disclosure

Authored with Claude Code (Opus 4.8).

Consolidate the standalone Harbor sandbox page into the Harbor
integrations evals doc. Remove src/langsmith/sandbox-harbor.mdx, add a
redirect from /langsmith/sandbox-harbor to
/langsmith/harbor-integrations#sandboxes, and point the Sandboxes-tab
sidebar entry and the Harbor card at the evals doc.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added the langsmith For docs changes to LangSmith label Jun 26, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Thanks for opening a docs PR, Nicholas Bohm (@Nickbohm555)! When it's ready for review, please add the relevant reviewers:

  • @katmayb or @fjmorris (LangSmith)

@github-actions

Copy link
Copy Markdown
Contributor

Mintlify preview branch generated: preview-nickbo-1782510823-4bedb54

Site preview: https://langchain-5e9cc07a-preview-nickbo-1782510823-4bedb54.mintlify.app

Important

Preview links may take a few minutes to start working while the deployment finishes.

Changed documentation pages (preview deep links):

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal langsmith For docs changes to LangSmith

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant