Skip to content

ci: add TypeScript quality profile - #22

Draft
TheDanniCraft wants to merge 4 commits into
masterfrom
ci/typescript-quality-profile
Draft

ci: add TypeScript quality profile#22
TheDanniCraft wants to merge 4 commits into
masterfrom
ci/typescript-quality-profile

Conversation

@TheDanniCraft

@TheDanniCraft TheDanniCraft commented Aug 3, 2026

Copy link
Copy Markdown
Member

📝 Description

Adds an opt-in TypeScript quality profile to the organization baseline. The
profile is independent of Supabase and applies to Bun-based TypeScript or
JavaScript repositories that explicitly enroll in it.

The central profile owns Biome, TypeScript, Vitest, Fallow, editor settings,
the Code Quality workflow, and its required ruleset. A new Repository Profile
Bootstrap workflow first creates the package-and-lockfile pull request for a
chosen repository. It adds only missing profile values, stops on a conflicting
repository-owned value, and regenerates bun.lock with lifecycle scripts
disabled. Once that PR is merged, normal baseline synchronization supplies the
remaining managed files and required check.

🧩 Type of Change

  • 🤖 CI/CD or automation
  • 🧹 Maintenance

🗺️ Areas

  • Infrastructure
  • Reliability

🧪 Testing and Validation

node --test .github/scripts/*.test.mjs
node --check .github/scripts/repository-baseline.mjs
node --check .github/scripts/profile-bootstrap.mjs
JSON schema parse checks
git diff --check

🔐 Security, Compatibility, and Operations

The profile is opt-in. The bootstrap workflow uses the existing organization
GitHub App token only for the selected Payload Bay repository, refuses package
contract conflicts, and runs bun install --lockfile-only --ignore-scripts.
It does not execute repository lifecycle scripts. Fallow review comments use
the normal pull-request token and remain bounded to ten comments.

📋 Checklist

  • I have read and understood the Contribution Guidelines
  • This pull request addresses one focused change
  • My pull-request title follows Conventional Commits
  • My commit messages follow Conventional Commits
  • I have performed a self-review of the complete diff
  • I understand and can explain every submitted change
  • I have removed debug output, unrelated changes, and generated noise
  • I have added or updated appropriate tests, or explained why none are needed
  • I have updated relevant documentation, or explained why none is needed
  • My changes introduce no unexpected warnings or errors
  • I have considered security, compatibility, deployment, and rollback effects
  • I have not included credentials, secrets, private payloads, or personal data
  • Any required dependent changes are linked and clearly identified

🤖 AI Usage

  • 🤖 This pull request includes AI-assisted work

🛠️ How AI Was Used

  • 🔍 Code review or analysis
  • 🧠 Ideation, research, or planning
  • ✍️ Drafting code
  • ♻️ Modifying or refactoring code
  • 📝 Drafting or editing documentation or other text
  • 🧪 Designing tests or validation steps
  • 🐛 Debugging or investigating a failure

🔎 AI Tools and Human Review

Codex assisted with the profile design and implementation. The resulting
configuration, synchronization logic, workflow, and tests were inspected
locally. The listed validation commands were run against the final worktree.

✅ AI Responsibility

  • I reviewed all AI-assisted content line by line
  • I verified factual, technical, security, and licensing claims
  • I tested the resulting behavior rather than trusting generated assertions
  • I accept full responsibility for the submitted work
  • I confirm this contribution is not automated spam or generic AI slop

💬 Additional Context

payloadbay/payloadbay is registered from the start, but its profile remains
inactive until the generated bootstrap PR has merged. Until then, baseline sync
reports the profile as waiting and does not apply its managed files or required
check. After merge, run baseline synchronization for payloadbay; it applies
the managed quality files before it adds the required status check.

@TheDanniCraft TheDanniCraft added the quality:exempt Maintainer override for automated pull request quality checks label Aug 3, 2026
@github-actions github-actions Bot added maintenance Refactoring, cleanup, tooling, or internal maintenance area:infrastructure Deployment, CI, hosting, and infrastructure area:reliability Queues, retries, recovery, and resilience labels Aug 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:infrastructure Deployment, CI, hosting, and infrastructure area:reliability Queues, retries, recovery, and resilience maintenance Refactoring, cleanup, tooling, or internal maintenance quality:exempt Maintainer override for automated pull request quality checks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant