Skip to content

docs(admin): add guide for enabling fullsend on private repositories - #1190

Merged
waynesun09 merged 4 commits into
mainfrom
feat/1188-private-repo-admin-guide
May 19, 2026
Merged

docs(admin): add guide for enabling fullsend on private repositories#1190
waynesun09 merged 4 commits into
mainfrom
feat/1188-private-repo-admin-guide

Conversation

@fullsend-ai-coder

Copy link
Copy Markdown
Contributor

defaults, AGENTS.md examples for preventing sensitive content reproduction, guardrail testing procedures, and deployment guidance based on data sensitivity.

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com


Closes #1188

Post-script verification

  • Branch is not main/master (feat/1188-private-repo-admin-guide)
  • Secret scan passed (gitleaks — 5f4e79e1610ae3ce8e8bcf95d530c1974ae6fcc1..HEAD)
  • Pre-commit hooks passed (authoritative run on runner)
  • Tests ran inside sandbox

Closes #1188. Covers public vs. private repo differences, agent safety
defaults, AGENTS.md examples for preventing sensitive content
reproduction, guardrail testing procedures, and deployment guidance
based on data sensitivity.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions

github-actions Bot commented May 19, 2026

Copy link
Copy Markdown

Site preview

Preview: https://f7dbfe8c-site.fullsend-ai.workers.dev

Commit: 58c23c58e9e1d9a335222b7f6b511187c087c9c0

…e requirements

Add a 'Supported scenarios' section near the top of the private
repositories guide that makes it clear which install mode to use for
private repos.

Per-repo install is self-contained — workflows live in the target repo
and call the public upstream actions directly, so there are no
cross-repo visibility constraints. This is the recommended mode for
private repos.

Per-org install uses cross-repo workflow_call into a shared .fullsend
config repo. When .fullsend is private, GitHub's access controls
silently reject workflow_call from any non-private caller — including
public repos and internal repos on Enterprise Cloud — producing 0 jobs
and no error message. Add a visibility matrix and call out the silent
failure explicitly.

Also tighten point 3 in "How private repos differ" to note that
the Actions log concern applies only to per-org install, since
per-repo install has no .fullsend config repo.

Signed-off-by: Wayne Sun <gsun@redhat.com>
@fullsend-ai-review

fullsend-ai-review Bot commented May 19, 2026

Copy link
Copy Markdown

Review

Findings

Low

  • [docs-currency] docs/guides/admin/installation.md — The installation guide discusses .fullsend config repo visibility constraints and private repository behavior (the paragraph noting that making .fullsend private causes silent failures for public/internal repos) but does not cross-reference the new private-repositories.md guide. Adding a "see also" link would help administrators who encounter the visibility warning during initial setup find the comprehensive private-repo guidance without searching.
    Remediation: Add a cross-reference after the visibility paragraph in installation.md pointing to private-repositories.md.

Info

  • [scope-note] docs/guides/admin/private-repositories.md — A contributor comment on Admin guide: enabling fullsend on private repositories #1188 requested coverage of indirect exposure paths — public repos with connections to private data (e.g., kustomize overlays, shared configs, cross-repo build dependencies). The guide partially addresses cross-visibility concerns (.fullsend config repo, retro agent) but does not cover the broader pattern of public repos inheriting private context. This could be addressed in a follow-up.
Previous run

Review

Findings

Info

  • [intent-alignment] docs/guides/admin/private-repositories.md — The guide covers all five areas requested in Admin guide: enabling fullsend on private repositories #1188 but does not address the indirect exposure paths raised in the contributor comment: cross-repo build dependencies, kustomize overlays, and shared configs that create indirect data exposure between private and public repos. This is a meaningful gap worth covering in a follow-up.
    Remediation: Add a section covering indirect exposure paths — scenarios where public repos with agentic workflows inherit access to private data through cross-repo dependencies (e.g., kustomize overlays, shared configs, cross-repo build inputs).

@@ -0,0 +1,187 @@
# Enabling fullsend on private repositories

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[info] intent-alignment

The guide covers all five areas requested in #1188 but does not address the indirect exposure paths raised in a contributor comment: cross-repo build dependencies, kustomize overlays, and shared configs that create indirect data exposure between private and public repos. This is a meaningful gap worth covering in a follow-up.

Suggested fix: Add a section covering indirect exposure paths — scenarios where public repos with agentic workflows inherit access to private data through cross-repo dependencies (e.g., kustomize overlays, shared configs, cross-repo build inputs).

@fullsend-ai-review

fullsend-ai-review Bot commented May 19, 2026

Copy link
Copy Markdown

Review follow-ups

Created follow-up issues for actionable non-blocking review findings:

  • #1223 — Installation guide discusses .fullsend visibility constraints for private repos but does not cross-reference the new private-repositories.md guide. A see-also link would help admins find comprehensive private-repo guidance.
  • #1224 — Contributor comment on Admin guide: enabling fullsend on private repositories #1188 requested coverage of indirect exposure paths (public repos with connections to private data). The guide partially covers cross-visibility concerns but does not address the broader pattern. Could be a follow-up.
Previous run

Review follow-ups

Created follow-up issues for actionable non-blocking review findings:

  • #1219 — The guide covers all five areas requested in Admin guide: enabling fullsend on private repositories #1188 but does not address the indirect exposure paths raised in a contributor comment: cross-repo build dependencies, kustomize overlays, and shared configs that create indirect data exposure between private and public repos. This is a meaningful gap worth covering in a follow-up.

Signed-off-by: Wayne Sun <gsun@redhat.com>
@fullsend-ai-review fullsend-ai-review Bot added ready-for-merge All reviewers approved — ready to merge and removed ready-for-merge All reviewers approved — ready to merge labels May 19, 2026
@waynesun09
waynesun09 added this pull request to the merge queue May 19, 2026
Merged via the queue into main with commit 7813c92 May 19, 2026
8 checks passed
@waynesun09
waynesun09 deleted the feat/1188-private-repo-admin-guide branch May 19, 2026 23:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-merge All reviewers approved — ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Admin guide: enabling fullsend on private repositories

1 participant