Skip to content

Align APR admin guide with current hooks product docs - #30

Closed
sankerr wants to merge 3 commits into
mainfrom
feature/align-apr-docs
Closed

sankerr wants to merge 3 commits into
mainfrom
feature/align-apr-docs

Conversation

@sankerr

@sankerr sankerr commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Overview

Align this plugin's Agent Package Resolution admin guide with the current hooks product docs. The Codex /hooks trust section is kept.

Details

  • Port the shared admin body from jfrog-agent-hooks docs/product/configure-agent-package-resolution.md (Windows profile paths, durable-off table, four-harness onboarding).
  • Keep the Codex install and hook-trust section after the setup summary.
  • Drop the hooks-only design-doc link that is not shipped in this repo.

Notes

This is docs-only. It does not bump the marketplace plugin version. The user guide already named Codex and is unchanged.

Co-authored-by: Cursor <cursoragent@cursor.com>
@sankerr
sankerr requested a review from a team as a code owner September 15, 2026 14:07
sankerr and others added 2 commits September 16, 2026 10:08
Windows support (jfrog-agent-hooks 365afa3) hasn't been released or
synced to this plugin's vendored modules/ yet (still pinned to
jfrog-agent-hooks/v0.12.1, which predates it). Documenting it now would
promise behavior the shipped plugin doesn't have. Re-add once a hooks
release including that commit is tagged and Sync Plugins bumps the pin.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Mirrors the same fix in jfrog-agent-hooks PR #145: the setup-summary sentence said plain "enabled: false" keeps APR off, contradicting the At a glance bullet two lines later, which correctly says the default file re-enables itself.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
sankerr added a commit that referenced this pull request Sep 16, 2026
Include the align-apr-docs PR on this sync branch. No extra Windows documentation added.
@sankerr

sankerr commented Sep 16, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by the v0.12.2 sync PR, which already includes these commits: #31

Closing this docs-only PR so review and merge happen on the sync PR (modules pin + APR docs together).

@sankerr

sankerr commented Sep 16, 2026

Copy link
Copy Markdown
Contributor Author

Closed: changes are on #31

@sankerr sankerr closed this Sep 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant