Skip to content

docs(governance): ratify organization policy and patterns#433

Merged
ss-o merged 12 commits into
mainfrom
feature-adr-ratify
Jun 1, 2026
Merged

docs(governance): ratify organization policy and patterns#433
ss-o merged 12 commits into
mainfrom
feature-adr-ratify

Conversation

@ss-o
Copy link
Copy Markdown
Member

@ss-o ss-o commented Jun 1, 2026

Summary

  • ratify and tighten organization ADRs, governance runbooks, and shared instructions
  • secure workflow configuration and add explicit concurrency conventions
  • move metrics, link-check, and deployment concurrency to workflow scope
  • pin the reusable Zsh CI setup action to an immutable organization revision
  • remove stale zsh-lint pattern references after the Go analyzer migration

Validation

  • workflow YAML parse and policy audit across all workspace workflow files
  • exact external action immutable-pin scan
  • git diff --check

ss-o added 12 commits May 29, 2026 04:35
Flip the four PROPOSED foundational ADRs (meta-repo pattern, zi as
canonical plugin manager, Conventional Commits, wiki content-root
boundaries) to ACCEPTED, since all are de-facto enforced across the
workspace. Record ss-o as decider and add a Decision Authority section
to the ADR runbook clarifying who may accept an ADR.
…Rs plus runbooks and instructions

Add ADR-0008 (branching model derived from ADR-0007 repo classes, resolving
the repos.yml branch-model drift at its source), ADR-0009 (testing/CI scope
by repo class), and ADR-0010 (security incident response: SLA, severity
targets, escalation, post-incident review). Add paired runbooks
(security-incident-response, onboarding, deprecation) and scoped instructions
(testing, documentation) operationalizing the new decisions.
…cking

Public repos have no private issues; replace the "private issue or maintainer
channel" intake step with GitHub repository Security Advisories (draft GHSA),
which provides private collaboration, a private fork, and CVE issuance. Also
generalize the incident-owner reference from a hardcoded handle to "security
contact (currently ss-o)".
ADR-0008: replace per-repo "use judgment" discretion with a canonical per-repo
branch-model table that repos.yml derives from, so drift is structurally
prevented; record updating CLAUDE.md as a post-acceptance action.
ADR-0009: label Conventional-Commits/trailer enforcement as target-state (no
live org-wide CI control exists yet), reference rather than restate workflow
conventions, and add dependency-scanning (ADR-0004) plus SAST for compiled
class-2 tools (ADR-0010).
ADR-0010: name GitHub Security Advisories as the private intake/tracking channel,
generalize the hardcoded owner to "security contact (currently ss-o)", and keep
the severity table canonical here (runbook now references it instead of dupe).
…t determinant

Re-verified every repo's branch model via git ls-remote: next exists only on
src, wiki, zi, zsh-lint, zsh-eza; all others are main-only. The prior table
wrongly listed zd (class 1) as next->main. zd deploys directly from main. This
also disproves "branch model follows the class": class 1 and class 2 are each
split across next and trunk, so the per-repo table is authoritative and the
ADR-0007 class is only an input (it sets the publication boundary and a default).
- Symlink AI instructions to central AGENTS.md to prevent fragmentation
- Apply IDE-agnostic terminology across instructions
- Ratify governance ADRs and align agent personas

Part of Epic ZSH-16.
@ss-o ss-o marked this pull request as ready for review June 1, 2026 04:12
@ss-o ss-o requested a review from a team as a code owner June 1, 2026 04:12
Copilot AI review requested due to automatic review settings June 1, 2026 04:12
@ss-o ss-o merged commit 544f83b into main Jun 1, 2026
4 of 5 checks passed
@ss-o ss-o deleted the feature-adr-ratify branch June 1, 2026 04:12
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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.

2 participants