Skip to content

Add Lapdog Uninstall section and sidebar nav entry#36823

Merged
Kyle-Verhoog merged 4 commits into
masterfrom
kyle.verhoog/llmobs-lapdog-uninstall-section
May 22, 2026
Merged

Add Lapdog Uninstall section and sidebar nav entry#36823
Kyle-Verhoog merged 4 commits into
masterfrom
kyle.verhoog/llmobs-lapdog-uninstall-section

Conversation

@Kyle-Verhoog
Copy link
Copy Markdown
Member

@Kyle-Verhoog Kyle-Verhoog commented May 19, 2026

What does this PR do? What is the motivation?

The Lapdog page shipped without a sidebar nav entry, so it was only reachable by search or direct link. It also had no on-page uninstall procedure, sending users to the GitHub README for teardown steps.

This PR:

  • Adds an Uninstall section to `content/en/llm_observability/lapdog.md` so the procedure (stop the agent, remove the Claude Code plugin and pi extension, delete `~/.lapdog`, then uninstall the package) lives on the docs page. The static dashboard's "Collecting sessions" popover deep-links to the new `#uninstall` anchor.
  • Adds a Lapdog entry to the LLM Observability sidebar in `config/_default/menus/main.en.yaml`, placed between Instrumentation and Monitoring. The following top-level entries (Monitoring, Evaluations, Experiments, MCP Server, Data Security and RBAC, Terms and Concepts, Guides) get their `weight` bumped by 1 to make room. Sub-entry weights are unchanged (they sort relative to their parent).

Merge instructions

Merge readiness:

  • Ready for merge

Additional notes

Vale was not run locally (binary not installed in my dev environment). Will rely on CI's Vale check.

Claude session: `1e26db38-e2e5-41d5-b214-b4a8c11b8ddb`
Resume: `claude --resume 1e26db38-e2e5-41d5-b214-b4a8c11b8ddb`

@Kyle-Verhoog Kyle-Verhoog requested a review from a team as a code owner May 19, 2026 15:43
@datadog-official

This comment has been minimized.

@github-actions
Copy link
Copy Markdown
Contributor

Preview links (active after the build_preview check completes)

Modified Files

Copy link
Copy Markdown
Contributor

@joepeeples joepeeples left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Content looks good, but the process should be a numbered procedure, not separate headings/sections for each step. Could you reformat the procedure? Thanks!

@Kyle-Verhoog Kyle-Verhoog changed the title [DOCS-TBD] Add Uninstall section to Lapdog page Add Uninstall section to Lapdog page May 21, 2026
@Kyle-Verhoog Kyle-Verhoog changed the title Add Uninstall section to Lapdog page [DOCS-TBD] Add Lapdog Uninstall section and sidebar nav entry May 21, 2026
@Kyle-Verhoog Kyle-Verhoog changed the title [DOCS-TBD] Add Lapdog Uninstall section and sidebar nav entry Add Lapdog Uninstall section and sidebar nav entry May 21, 2026
@Kyle-Verhoog Kyle-Verhoog force-pushed the kyle.verhoog/llmobs-lapdog-uninstall-section branch from 5433bb6 to 36514da Compare May 21, 2026 16:59
@github-actions github-actions Bot added the Architecture Everything related to the Doc backend label May 21, 2026
@Kyle-Verhoog Kyle-Verhoog merged commit dd824b5 into master May 22, 2026
19 checks passed
@Kyle-Verhoog Kyle-Verhoog deleted the kyle.verhoog/llmobs-lapdog-uninstall-section branch May 22, 2026 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Architecture Everything related to the Doc backend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants