GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — chore: Remove deprecated `src teams` commands by marcleblanc2 · Pull Request #1376 · sourcegraph/src-cli

Conversation

@marcleblanc2

@marcleblanc2 marcleblanc2 commented Sep 9, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Deletes the deprecated src teams command from src cli and the generated docs

Test plan

  • go test ./cmd/src/... passes
  • go run ./cmd/src teams reports an unknown subcommand
  • go run ./cmd/src doc -o /tmp/out && find /tmp/out -name '*team*' prints nothing

Stack

Merge in order (FE-502):

  1. src-cli#1375 src doc: subcommand pages for debug/search-jobs/snapshot, fix root index
  2. src-cli#1376 remove src teams
  3. src-cli#1377 generate src help from registered commands
  4. src-cli release containing the above
  5. sourcegraph#15528 bump the src-cli pin, check in the new src doc output
  6. sourcegraph#15529 make the docs sync mirror doc/cli/references instead of copying over it

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
This was referenced Sep 9, 2026
docs/cli: Add missing doc pages for src-cli commands sourcegraph/docs#1889
Merged
fix/doc: Add missing commands to helper text #1375
Merged
fix/help: Generate help command list from registered commands #1377
Merged
@marcleblanc2 marcleblanc2 changed the title remove: src teams commands chore/remove: src teams commands Sep 9, 2026
@marcleblanc2 marcleblanc2 changed the title chore/remove: src teams commands chore: Remove deprecated src teams commands Sep 9, 2026
marcleblanc2 added a commit to sourcegraph/docs that referenced this pull request Sep 11, 2026
@marcleblanc2 @ampagent
docs/cli: Add missing doc pages for src-cli commands (#1889)
b6a21bb
Part of [FE-502](https://linear.app/sourcegraph/issue/FE-502). **Step 4
of 4** in a cross-repo stack, but **mergeable now** — it pre-seeds what
the generated-docs sync will eventually write, so the live reference
gets fixed without waiting on a src-cli release.

## Why

The `src` CLI reference is generated by `src doc` and synced here by
sourcegraph/sourcegraph's `sync/generated-docs` job. Two generator bugs
(fixed in sourcegraph/src-cli#1375) left this reference incomplete:

- `search-jobs`, `debug` and `snapshot` were never registered in the
generator's `commanders` map, so each is a single page with only the
group help; their 16 subcommands have no reference pages. (The 8
`search-jobs/*.mdx` pages here were hand-written by Travis Lyons in May
2025 to paper over this; nothing links to them.)
- `index.mdx` lists only the 8 urfave/cli commands (`abc`, `api`,
`auth`, `codeowners`, `login`, `orgs`, `users`, `version`) — `batch`,
`repos`, `search`, `config`, etc. are missing from
https://sourcegraph.com/docs/cli/references today.

## What

- `index.mdx`: lists all 19 top-level commands.
- `debug/{index,compose,kube,server}.mdx`,
`snapshot/{index,databases,restore,summary,test,upload}.mdx`: new.
- `search-jobs/index.mdx`: new;
`search-jobs/{cancel,create,delete,get,list,logs,restart,results}.mdx`:
hand-written pages replaced by generated ones (same usage text, plus a
flags table; the `<p className="subtitle">` blurbs go away since the
generator doesn't emit them).
- Deleted `debug.mdx`, `search-jobs.mdx`, `snapshot.mdx`.
- Deleted `teams.mdx` and dropped `teams` from `index.mdx`: teams were
removed in Sourcegraph 7.0 and sourcegraph/src-cli#1376 removes the
command, so the generator no longer emits this page. No redirect (same
call as #1886). The sync job never deletes files, and in contentlayer
routing a flat `foo.mdx` shadows `foo/index.mdx` (the same bug #1886
fixed for `auth.mdx`/`codeowners.mdx`), so these have to go by hand for
the new index pages to be reachable.

Content was produced by running the patched generator (src-cli `main` +
#1375) and the same `tools/md2mdx` conversion the sync uses, so the sync
PR that follows sourcegraph/sourcegraph#15528 should be a no-op for
these paths.

No redirects: the three deleted URLs
(`/cli/references/{debug,search-jobs,snapshot}`) keep resolving, now to
the new index pages.

## Stack

1. sourcegraph/src-cli#1375 — generator fix + tests;
sourcegraph/src-cli#1376 (stacked) — remove `src teams`;
sourcegraph/src-cli#1377 (stacked) — `src help` generated from
registered commands, tests help == docs
2. src-cli 7.7.0 release (none since 7.6.0; blocks 3)
3. sourcegraph/sourcegraph#15528 — pin bump + `OUTPUT_FILES` +
regenerate (draft until 2)
4. **this PR**
5. sourcegraph/sourcegraph#15529 — make the docs sync mirror
`docs/cli/references/` so removed commands disappear automatically
(merge after 3 and this PR)

Related: #1886 (removed 28 stale pages for commands that no longer
exist).

## Test plan

`npx contentlayer build` → 528 documents; routes resolve to the intended
files:

```
/cli/references/debug        <- cli/references/debug/index.mdx
/cli/references/search-jobs  <- cli/references/search-jobs/index.mdx
/cli/references/snapshot     <- cli/references/snapshot/index.mdx
```
plus the 16 subcommand routes. (The `ERR_INVALID_ARG_TYPE`/clipanion
stack trace during the build is pre-existing on `main`.)

## Amp threads

- [Stale command
docs](https://ampcode.com/threads/T-01a08410-86ca-72be-9928-2810e837fae1)

---------

Co-authored-by: Amp <[email protected]>
@marcleblanc2
marcleblanc2 added this pull request to stack #1380 September 25, 2026 04:21
@marcleblanc2
marcleblanc2 removed this pull request from stack #1380 September 25, 2026 04:22
BolajiOlajide
BolajiOlajide approved these changes Sep 28, 2026
View reviewed changes
Base automatically changed from fix/doc-legacy-groups to main September 28, 2026 09:38
@github-actions

github-actions Bot commented Sep 28, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

Open the Diff Tour for this PR in Sourcegraph

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

marcleblanc2 and others added 2 commits September 28, 2026 19:17
@marcleblanc2 @ampagent
remove: src teams commands
ecc526b
Teams were removed in Sourcegraph 7.0. Since #1257 (src-cli 7.0.0) the
command has refused to run against 7.x instances, so it only works against
instances that are no longer supported. Remove it and its reference page.

Drops the "teams.md" entry from the doc golden list and the
legacyGroupsWithoutSubcommandPages exemption added in #1375, so every
remaining legacy group is now expanded into subcommand pages.

Part of https://linear.app/sourcegraph/issue/FE-502

Amp-Thread-ID: https://ampcode.com/threads/T-01a08410-86ca-72be-9928-2810e837fae1
Co-authored-by: Amp <[email protected]>
@marcleblanc2 @ampagent
remove Sourcegraph70 feature flag and stale extensions/gateway help e…
92beb80
…ntries

Sourcegraph70 was only used by the teams version gate. The `extensions` and
`gateway` commands were removed in #1173 and earlier but were still listed in
`src help`.

Part of https://linear.app/sourcegraph/issue/FE-502

Amp-Thread-ID: https://ampcode.com/threads/T-01a08410-86ca-72be-9928-2810e837fae1
Co-authored-by: Amp <[email protected]>
@marcleblanc2
marcleblanc2 force-pushed the remove-teams-command branch from 89e7de3 to 92beb80 Compare September 28, 2026 19:19
@marcleblanc2
marcleblanc2 merged commit 4c967dd into main Sep 28, 2026
9 checks passed

Uh oh!

There was an error while loading. Please reload this page.

@marcleblanc2
marcleblanc2 deleted the remove-teams-command branch September 28, 2026 19:27
@marcleblanc2 marcleblanc2 mentioned this pull request Nov 11, 2026
chore: Remove the src debug server command #1407
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@BolajiOlajide BolajiOlajide BolajiOlajide approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@marcleblanc2 @BolajiOlajide
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.