Skip to content

docs: fix DefaultAgentDownloadURL file path - #962

Merged
skevetter merged 1 commit into
mainfrom
docs-keeper/default-agent-url-path
Aug 9, 2026
Merged

skevetter merged 1 commit into
mainfrom
docs-keeper/default-agent-url-path

Conversation

@devsy-app

@devsy-app devsy-app Bot commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Doc area

CONTRIBUTING.md — "Development Setup > Initial Setup" (the "change Devsy agent code" step).

Drift found

The step linked DefaultAgentDownloadURL to ./pkg/agent/agent.go:

Exchange the URL in DefaultAgentDownloadURL with a custom public repository release

That file path is wrong. The DefaultAgentDownloadURL function is defined in ./pkg/config/repo.go, not ./pkg/agent/agent.go. The link was stale and pointed at a file that does not contain the referenced symbol.

Correction

Updated the link target from ./pkg/agent/agent.go to ./pkg/config/repo.go.

Code verification performed

  • grep -rn "func DefaultAgentDownloadURL" pkg/ → defined at pkg/config/repo.go:47.
  • Confirmed pkg/agent/agent.go does not define DefaultAgentDownloadURL (it only calls config.DefaultAgentDownloadURL() at line 442).
  • Confirmed pkg/config/repo.go exists and contains the function plus the related AgentDownloadBaseURL / AgentLatestDownloadURL constants.

Validation

  • task cli:format — clean.
  • task cli:lint:ci — 0 issues.
  • task cli:test — only the known pre-existing pkg/git TestRepoClone* failures (unchanged baseline; this PR does not touch pkg/git or any Go source).

This PR was created by an AI agent as part of an automated daily docs job.

## Doc area

`CONTRIBUTING.md` — "Development Setup > Initial Setup" (the "change Devsy agent code" step).

## Drift found

The step linked `DefaultAgentDownloadURL` to `./pkg/agent/agent.go`:

> Exchange the URL in [DefaultAgentDownloadURL](./pkg/agent/agent.go) with a custom public repository release

That file path is wrong. The `DefaultAgentDownloadURL` function is defined in `./pkg/config/repo.go`, not `./pkg/agent/agent.go`. The link was stale and pointed at a file that does not contain the referenced symbol.

## Correction

Updated the link target from `./pkg/agent/agent.go` to `./pkg/config/repo.go`.

## Code verification performed

- `grep -rn "func DefaultAgentDownloadURL" pkg/` → defined at `pkg/config/repo.go:47`.
- Confirmed `pkg/agent/agent.go` does not define `DefaultAgentDownloadURL` (it only calls `config.DefaultAgentDownloadURL()` at line 442).
- Confirmed `pkg/config/repo.go` exists and contains the function plus the related `AgentDownloadBaseURL` / `AgentLatestDownloadURL` constants.

## Validation

- `task cli:format` — clean.
- `task cli:lint:ci` — 0 issues.
- `task cli:test` — only the known pre-existing `pkg/git` `TestRepoClone*` failures (unchanged baseline; this PR does not touch `pkg/git` or any Go source).

This PR was created by an AI agent as part of an automated daily docs job.
@netlify

netlify Bot commented Aug 9, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for devsydev canceled.

Name Link
🔨 Latest commit 5c9b87d
🔍 Latest deploy log https://app.netlify.com/projects/devsydev/deploys/6a7875900ff9840008ae913b

@netlify

netlify Bot commented Aug 9, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for images-devsy-sh canceled.

Name Link
🔨 Latest commit 5c9b87d
🔍 Latest deploy log https://app.netlify.com/projects/images-devsy-sh/deploys/6a787590ede77f00087ae576

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

AI Reviewer: run a review on demand. To trigger the first review automatically, go to your organization or repository integration settings. AI can make mistakes. Always validate suggestions.

Run reviewer

TIP This summary will be updated as you push new changes.

@skevetter
skevetter marked this pull request as ready for review August 9, 2026 16:59
@skevetter
skevetter merged commit 57ef724 into main Aug 9, 2026
26 checks passed
@skevetter
skevetter deleted the docs-keeper/default-agent-url-path branch August 9, 2026 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant