Skip to content

feat: adopt weekly template-sync with the tenant-template ownership model#180

Merged
devantler merged 3 commits into
mainfrom
claude/template-sync-176
Jul 17, 2026
Merged

feat: adopt weekly template-sync with the tenant-template ownership model#180
devantler merged 3 commits into
mainfrom
claude/template-sync-176

Conversation

@devantler

Copy link
Copy Markdown
Contributor

🤖 Generated by the Daily AI Engineer

Why

Repos created from this template never receive template improvements — this week's portfolio-wide CI fixes reached instances only by hand-backporting. The tenant template already solved this with a proven sync + ownership model.

What

Adopts the weekly template-sync workflow (skipped in this repo, opens a sync PR in instances) plus a default .templatesyncignore encoding the three settled ownership classes — template-owned plumbing, instance-owned code/identity, scaffold-time-only machinery — documented in the README. One deliberate deviation from the tenant template: ci.yaml stays template-owned here, since every Go instance shares the same required-checks aggregation.

Fixes #176. The .NET twin (devantler-tech/dotnet-template#291) ports this once proven.

…odel

Fixes #176

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@devantler

Copy link
Copy Markdown
Contributor Author

🤖 Generated by the Daily AI Engineer

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 17139005ac

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/template-sync.yaml Outdated
Comment thread .templatesyncignore Outdated
Comment thread .templatesyncignore Outdated
…ses (codex review)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@devantler

Copy link
Copy Markdown
Contributor Author

🤖 Generated by the Daily AI Engineer

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2b8743f00a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .templatesyncignore Outdated
…owned (codex round 2)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@devantler

Copy link
Copy Markdown
Contributor Author

🤖 Generated by the Daily AI Engineer

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Keep it up!

Reviewed commit: ef5131fc6b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@devantler

Copy link
Copy Markdown
Contributor Author

🤖 Generated by the Daily AI Engineer

Readiness record for head ef5131f (self-promotion):

  1. Programmatically tested — full CI green at head (required aggregate + CodeQL + zizmor + Validate Scaffold); actionlint + zizmor clean locally on the new workflow; pentad clear (0 threads, CLEAN, codex lane).
  2. Reviewed — Codex clean pass at exactly this head (Reviewed commit: ef5131fc6b), after two findings rounds (4 findings fixed: credential-less-instance guard, pkg/ wholesale protection, .golangci.yml instance-owned, header-comment consistency).
  3. Tried as a user — this change has no exercisable runtime surface in the template repo itself: the sync job is deliberately if-guarded off here and enacts only in instances. Nearest observation: the enacting path (devantler-tech/actions template-sync reusable workflow consuming .templatesyncignore) is the identical, live-proven config running in gitops-tenant-template's instances, which delivered three plumbing updates this month (Template-sync for instances — shared plumbing only, tenant-template ownership model #176's evidence); the workflow parses clean and the if-guard was verified by this PR's own CI (job skipped in-template as designed). First real instance run is observable after merge on any go-template instance's Monday schedule.

Promoting and merging per the merge policy.

@devantler
devantler marked this pull request as ready for review July 17, 2026 07:01
@devantler
devantler requested a review from a team as a code owner July 17, 2026 07:01
@devantler
devantler merged commit 93a70c3 into main Jul 17, 2026
11 checks passed
@devantler
devantler deleted the claude/template-sync-176 branch July 17, 2026 07:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Template-sync for instances — shared plumbing only, tenant-template ownership model

1 participant