Skip to content

docs: remove local-dev.md, consolidate into running-agents-locally#2179

Open
rh-hemartin wants to merge 1 commit into
mainfrom
fix/762-local-dev
Open

docs: remove local-dev.md, consolidate into running-agents-locally#2179
rh-hemartin wants to merge 1 commit into
mainfrom
fix/762-local-dev

Conversation

@rh-hemartin

Copy link
Copy Markdown
Member

Summary

  • Removes docs/guides/dev/local-dev.md, which duplicated most of running-agents-locally.md
  • Moves the "build from source" step (make go-build) to CONTRIBUTING.md
  • Updates all inbound links to point to running-agents-locally.md or CONTRIBUTING.md
  • Fixes a pre-existing broken link (-inference.mdgetting-inference.md) in docs/guides/README.md

Closes #762.

Test plan

  • make lint passes
  • All links from updated files resolve correctly

🤖 Generated with Claude Code

local-dev.md duplicated most of running-agents-locally.md. Its unique
content (build from source) moved to CONTRIBUTING.md; cross-platform
binary internals were already covered by cli-internals.md. All inbound
links updated. Closes #762.

Signed-off-by: Hector Martinez <hemartin@redhat.com>
@github-actions

Copy link
Copy Markdown

Site preview

Preview: https://77a4cf38-site.fullsend-ai.workers.dev

Commit: d1c6644196b0baa31b2958459613c8e45a4462a8

@fullsend-ai-review

fullsend-ai-review Bot commented Jun 11, 2026

Copy link
Copy Markdown

🤖 Finished Review · ✅ Success · Started 2:57 PM UTC · Completed 3:09 PM UTC
Commit: d1c6644 · View workflow run →

@codecov

codecov Bot commented Jun 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@fullsend-ai-review

Copy link
Copy Markdown

Review

Findings

Medium

  • [scope-mismatch] docs/guides/dev/local-dev.md — The PR claims to close issue docs: add SA key creation steps to local-dev guide #762, which asked to add SA key creation steps to local-dev.md. The PR instead deletes local-dev.md entirely. The surviving running-agents-locally.md already contains the SA key gcloud commands (lines 64–74), so the underlying need is met, but the PR description should clarify this reasoning rather than implying the deletion itself resolves the issue.

  • [audience-classification-violation] CONTRIBUTING.md:42 — Build-from-source instructions are placed in CONTRIBUTING.md, which otherwise covers social norms and PR conventions. ADR-0023 leaves docs/guides/dev/ unchanged but doesn't prescribe content placement, so this isn't a hard violation — but a docs/guides/dev/ placement or a section in cli-internals.md would be more consistent with how other technical contributor content is organized.

  • [information-loss] docs/guides/dev/local-dev.md — The deleted file contains a "Security scanning and cross-platform binary" section with a binary resolution priority table (4 strategies: --fullsend-binary, GitHub Release download, cross-compile, latest release fallback), ELF validation behavior, and cross-compilation instructions. running-agents-locally.md mentions FULLSEND_SANDBOX_ARCH (line 287) and --fullsend-binary (line 311) only as brief troubleshooting tips, not with the detailed strategy context. Consider preserving this content in running-agents-locally.md or noting that it's intentionally dropped.

Low

  • [cross-reference-format] docs/guides/user/running-agents-locally.md:7 — The updated callout uses a quoted section name (see the "Building from source" section in CONTRIBUTING.md) instead of a clickable markdown link. Every other cross-reference in the changed files uses markdown link syntax. Consider: see [Building from source](../../CONTRIBUTING.md#building-from-source).

  • [cross-reference-completeness] docs/guides/dev/cli-internals.md:3,533 — Both cross-references in this developer-focused doc now point to the user-facing running-agents-locally.md. Since CONTRIBUTING.md now has the build instructions, consider adding a reference to that as well for developer readers.

Info

  • [broken-link-fix] docs/guides/README.md — The PR description mentions fixing a pre-existing broken link (-inference.mdgetting-inference.md), but this change is not visible in the diff. The only change to docs/guides/README.md is removing the local-dev.md link.

@fullsend-ai-review fullsend-ai-review Bot added the requires-manual-review Review requires human judgment label Jun 11, 2026
@rh-hemartin rh-hemartin self-assigned this Jun 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

requires-manual-review Review requires human judgment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: add SA key creation steps to local-dev guide

2 participants