Skip to content

docs(#2071): add public hosted mint URL to mint administration guide - #2072

Merged
rh-hemartin merged 2 commits into
mainfrom
agent/2071-add-public-mint-url
Jun 10, 2026
Merged

docs(#2071): add public hosted mint URL to mint administration guide#2072
rh-hemartin merged 2 commits into
mainfrom
agent/2071-add-public-mint-url

Conversation

@fullsend-ai-coder

Copy link
Copy Markdown
Contributor

Add a "Hosted mint" section before Prerequisites that states the public hosted mint URL plainly, so users who don't need to self-host can grab the URL and move on without hunting through CLI flag docs or source code.

Also update the introductory callout to remove the "work is in progress" language, since the hosted mint service is operational (referenced in #1145 and #1184).

Note: make lint could not run due to Go module cache permission errors in the sandbox (unrelated to this docs-only change). No Go code was modified.


Closes #2071

Post-script verification

  • Branch is not main/master (agent/2071-add-public-mint-url)
  • Secret scan passed (gitleaks — 46eb2347412401912817b8c5e04ebf4b6ce761b2..HEAD)
  • Pre-commit hooks passed (authoritative run on runner)
  • Tests ran inside sandbox

Add a "Hosted mint" section before Prerequisites that states the
public hosted mint URL plainly, so users who don't need to self-host
can grab the URL and move on without hunting through CLI flag docs
or source code.

Also update the introductory callout to remove the "work is in
progress" language, since the hosted mint service is operational
(referenced in #1145 and #1184).

Note: `make lint` could not run due to Go module cache permission
errors in the sandbox (unrelated to this docs-only change). No Go
code was modified.

Closes #2071
@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown

Site preview

Preview: https://6819d1ad-site.fullsend-ai.workers.dev

Commit: c2ae5b905a0d481e3e8a3603d1a302e2abece557

@fullsend-ai-review

fullsend-ai-review Bot commented Jun 9, 2026

Copy link
Copy Markdown

🤖 Finished Review · ✅ Success · Started 12:35 PM UTC · Completed 12:43 PM UTC
Commit: ba204cb · View workflow run →

@rh-hemartin

Copy link
Copy Markdown
Member

/fs-fix add the URL to the github-setup.md as well

@fullsend-ai-coder

fullsend-ai-coder Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor Author

🤖 Finished Fix · ✅ Success · Started 12:41 PM UTC · Completed 12:45 PM UTC
Commit: ba204cb · View workflow run →

@fullsend-ai-review

fullsend-ai-review Bot commented Jun 9, 2026

Copy link
Copy Markdown

Review

Findings

Medium

  • [stale-reference] docs/guides/getting-started/installation.md:38 — The Mint service note still states "Work is in progress to offer this as a secure, trusted public service" but this PR removes that same language from mint-administration.md and adds a "Hosted mint" section with the actual public URL, indicating the service is operational. This leaves contradictory documentation across the two files.
    Remediation: Update line 38 in installation.md to reflect that the hosted mint service is now available, and optionally link to the Hosted mint section in mint-administration.md.

Low

  • [code-block-formatting] docs/guides/infrastructure/mint-administration.md:10 — Code block containing the hosted mint URL lacks a language specifier. All 11 existing code blocks in the file use bash; this new block should use text for consistency with the file's convention of always specifying a language.

Info

  • [document-structure] docs/guides/infrastructure/mint-administration.md:7 — The "Hosted mint" section is positioned before "Prerequisites," which differs from typical guide structure but has editorial justification: users who can use the hosted mint can skip the rest of the guide entirely.

  • [authorization-clarity] docs/guides/infrastructure/mint-administration.md:8 — Issue docs: add public mint URL to top of mint administration guide #2071 requested documenting the public mint URL near the top of the guide. The implementation satisfies the stated requirement. Scope is authorized and matches the tier (documentation enhancement).

Previous run

Review

Findings

Low

  • [code-block-formatting] docs/guides/infrastructure/mint-administration.md:10 — Code block containing the hosted mint URL lacks a language specifier. The existing file uses bash for all 11 code blocks; this new block should use text for consistency.

Info

  • [document-structure] docs/guides/infrastructure/mint-administration.md:7 — The "Hosted mint" section is positioned before "Prerequisites," which is a defensible editorial choice (users who can use the hosted mint can skip the rest of the guide), but differs from the typical guide structure of audience → prerequisites → setup steps.

## Hosted mint

The fullsend team operates a public hosted mint service. If your organization is enrolled, you can use it directly without deploying your own:

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[low] code-block-formatting

Code block containing the hosted mint URL lacks a language specifier. The existing file uses bash-tagged code blocks for all 11 code blocks; this new block should use text for consistency.

Suggested fix: Change the opening fence to ```text

> **This guide is for platform operators** who deploy, manage, or troubleshoot the token mint Cloud Function. If you are an end user setting up fullsend for your organization, see [Installing fullsend](../getting-started/installation.md) instead — the mint is typically deployed once by a platform operator, and organizations are enrolled as needed. Work is in progress to offer a hosted public mint service, which will further reduce the need for per-org mint administration.
> **This guide is for platform operators** who deploy, manage, or troubleshoot the token mint Cloud Function. If you are an end user setting up fullsend for your organization, see [Installing fullsend](../getting-started/installation.md) instead — the mint is typically deployed once by a platform operator, and organizations are enrolled as needed.

## Hosted mint

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] document-structure

The Hosted mint section is positioned before Prerequisites, which differs from the typical guide structure but is a defensible editorial choice.

Add the public hosted mint URL inline in the prerequisites section of the
GitHub setup guide, with a cross-reference to the Hosted mint section in
the mint administration guide. This ensures users find the URL in both
relevant guides.

Addresses human review feedback on #2072

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@fullsend-ai-review fullsend-ai-review Bot added the ready-for-merge All reviewers approved — ready to merge label Jun 9, 2026
@fullsend-ai-coder

Copy link
Copy Markdown
Contributor Author

🔧 Fix agent — iteration 1 (human-triggered)

Added the hosted mint URL to the github-setup.md prerequisites section per human instruction, matching the URL already added to mint-administration.md in this PR.

Fixed (1):

  1. add hosted mint URL to github-setup.md (docs/guides/getting-started/github-setup.md): Added the public hosted mint URL inline in the prerequisites section where the --mint-url flag is documented, with a cross-reference link to the Hosted mint section in mint-administration.md

Tests: passed

Updated by fullsend fix agent

@fullsend-ai-review

fullsend-ai-review Bot commented Jun 9, 2026

Copy link
Copy Markdown

🤖 Finished Review · ✅ Success · Started 12:46 PM UTC · Completed 12:55 PM UTC
Commit: ba204cb · View workflow run →

@fullsend-ai-review fullsend-ai-review Bot added requires-manual-review Review requires human judgment and removed ready-for-merge All reviewers approved — ready to merge labels Jun 9, 2026
@rh-hemartin
rh-hemartin added this pull request to the merge queue Jun 10, 2026
Merged via the queue into main with commit edc5f21 Jun 10, 2026
9 checks passed
@rh-hemartin
rh-hemartin deleted the agent/2071-add-public-mint-url branch June 10, 2026 07:05
@fullsend-ai-retro

fullsend-ai-retro Bot commented Jun 10, 2026

Copy link
Copy Markdown

🤖 Finished Retro · ✅ Success · Started 7:06 AM UTC · Completed 7:11 AM UTC
Commit: 4ed6da4 · View workflow run →

@fullsend-ai-retro

Copy link
Copy Markdown

Retro: PR #2072 — Add public mint URL to mint administration guide

Workflow timeline: Issue #2071 was filed by ralphbean at 12:22Z. The triage agent finished in 2 minutes, the code agent opened PR #2072 within 11 minutes, and the review agent posted findings by 12:35Z. A human requested an additional change via /fs-fix at 12:39Z, and the fix agent committed within 4 minutes. Human approval came the next morning, and the PR was merged at 07:05Z. The automated pipeline was fast and efficient.

Key observation — unresolved review finding: The review agent flagged a medium-severity stale reference in installation.md:38 (still says the hosted mint is "work in progress"), but this was never addressed. The fix agent was dispatched by the human for a different task (adding the URL to github-setup.md), and the stale reference was not part of that scope. The human approved without addressing it either. The stale language remains in the file post-merge.

No new proposals — existing issues cover the gaps:

  • #870 — Medium findings should trigger fix agent or block merge (directly applicable here)
  • #1941 — Fix agent should address all medium+ findings before declaring completion
  • #1794 — Code agent should grep-verify completeness after rename/terminology tasks (the code agent updated "work in progress" language in mint-administration.md but didn't search for the same pattern in other files)
  • #1956 — Auto-file tracking issues for unresolved medium+ findings (would have caught this post-merge)

This PR provides another data point validating the need for #870 and #1941 in particular. The stale reference in installation.md:38 remains unfixed and should be addressed separately.

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 public mint URL to top of mint administration guide

1 participant