Content review: content/docs/iac/concepts/inputs-outputs/all.md - #20618
Content review: content/docs/iac/concepts/inputs-outputs/all.md#20618pulumi-bot wants to merge 1 commit into
Conversation
Automated existing-content review. Co-Authored-By: Claude <noreply@anthropic.com>
|
Your site preview for commit 1414167 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-20618-14141675.s3-website.us-west-2.amazonaws.com Changed pages: |
|
@claude #new-review |
Pre-merge Review — Last updated 2026-07-31T18:35:54ZTip Summary: This is an automated content-review PR touching one docs page, Review confidence:
Investigation log
🔍 Verification trail3 claims extracted · 3 verified · 0 unverifiable · 0 contradicted
🚨 Outstanding in this PRThese must be resolved or refuted before merging.
|
|
🤖 Review regenerated on @CamSoper's request. |
Important
This PR is set to auto-merge (squash). Once it has an approving review and the required build check passes, GitHub will merge it automatically — approving this PR will merge it. To prevent that, disable auto-merge (or convert the PR back to a draft) before approving.
Why this page
content/docs/iac/concepts/inputs-outputs/all.md→ /docs/iac/concepts/inputs-outputs/all/no_retire: true)CLICKSTREAM.FCT_PAGEVIEWS)attempts: 0)This section is composed deterministically from the selection queue; do not edit it.
Fixes applied
No pre-stubbed candidates:
.verified-claims.jsonreturned 0 contradicted/mismatch verdicts and no Vale finding carrieddeterministic_fix: true. The fixes below are redirected internal links found during the review — every one resolves today only via analiases:entry on the destination page, so each is a de-aliasing to the canonical full path with no change in destination.[apply](/docs/concepts/inputs-outputs/apply/)is an alias path, not the canonical URLcontent/docs/iac/concepts/inputs-outputs/apply.mddeclares/docs/concepts/inputs-outputs/apply/underaliases:; its canonicalrendered_urlis/docs/iac/concepts/inputs-outputs/apply//docs/iac/concepts/inputs-outputs/apply/applylink/docs/iac/concepts/inputs-outputs/apply/[Using output helpers](/docs/concepts/inputs-outputs/helpers/)is an alias pathcontent/docs/iac/concepts/inputs-outputs/helpers.mddeclares/docs/concepts/inputs-outputs/helpers/underaliases:/docs/iac/concepts/inputs-outputs/helpers/[stack output](/docs/using-pulumi/stack-outputs-and-references/)is an alias path; no page owns that URL directlycontent/tutorials/stack-outputs-refs-aws/index.mddeclares/docs/using-pulumi/stack-outputs-and-references/underaliases:(this page was its only remaining inbound reference)/tutorials/stack-outputs-refs-aws/— the same destination the redirect already servedFindings not applied
deterministic_fix; recasting the sentence into imperative second person means authoring new prose (“Suppose you have created…” vs. restructuring the whole scenario setup), which is a judgment call, not a substitution.allfunction can also be used…” — nag only, and the phrase is doing real work here (it links this section back to the preceding “Creating a new string” section); the sentence also carries a redundant “In addition … also” that a human may want to collapse. Left for an editorial pass.ApplyTcallback is declared asfunc (args []interface{}) pulumi.StringOutput, i.e. it returns anOutputfrom insideApplyTrather than a plainstring, soconnectionStringwould be an output-of-output. Claimc11verified only at medium confidence (the SDK doc comment it was checked against uses the same shape), and correcting it means rewriting the snippet's signature and body — above the high-confidence bar. Worth a human look.For the judgment-level items above, run
/glow-up content/docs/iac/concepts/inputs-outputs/all.md.Screenshot check
No images. The page source references no screenshots, diagrams, or other content images (only the generic shared
meta_imagecard, if any), so there is nothing to verify. (Determined from the source; the screenshot pass was skipped.)Rendered content
Ran (
make build→public/docs/iac/concepts/inputs-outputs/all/index.html). Two content-bearing shortcodes on this page:pulumi-apply— renders only the per-language spelling of the apply method name (apply/Apply). No checkable factual residue.example-program path="aws-s3bucket-bucketobject-interpolate"— injects the six-language interpolation program fromstatic/programs/aws-s3bucket-bucketobject-interpolate/. Residue claims checked: TypeScriptpulumi.concat/pulumi.interpolate, PythonOutput.concat/Output.format, Gopulumi.Sprintf, C#Output.Format, JavaOutput.format, and the YAML${...}interpolation — all are current, correctly named helpers, consistent with the surrounding prose ("build an S3 URL from bucket output values using each language's interpolation helper") and with Using output helpers. The Go tab pinspulumi-aws/sdk/v7, matching the checked-in program. No residue findings; no shared-source (shortcode/partial/data) edits made, so nothing here is multi-page.Environment note: this checkout needed
yarn install(root +theme/+theme/stencil/) andnode scripts/build-icon-sprite.jsbeforemake buildwould get as far as Hugo, and the full build then reported errors on unrelated pages elsewhere in the corpus. The reviewed page itself rendered cleanly, which is what this pass inspects.Verification
make lint: ✅make lintre-verified by the workflow on141416756b.verified-claims.json: 27 verdict(s); 0 contradicted/mismatch, 0 unverifiable.vale-findings.json: 3 finding(s).readthrough-findings.json: ran=True, 0 finding(s).frontmatter-validation.json: 1 file(s); 0 alias collision(s)