Skip to content

chore: clean up repository configuration - #205

Merged
maxktz merged 6 commits into
mainfrom
feat/cleanup
Sep 16, 2026
Merged

maxktz merged 6 commits into
mainfrom
feat/cleanup

Conversation

@maxktz

@maxktz maxktz commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Remove obsolete repository files and simplify the remaining build configuration.

  • remove the local PayKit architecture skill and .nvmrc
  • pin Node 22 directly in workflows that previously read .nvmrc
  • inline the PayKit tsdown configuration and remove its single-use root helper
  • remove the unused lint-staged dependency
  • remove the contributing guide and completed release-backfill notes

Validation:

  • pnpm --filter paykitjs build
  • pnpm exec oxlint --deny-warnings -- packages/paykit/tsdown.config.ts
  • pnpm exec oxfmt --check packages/paykit/tsdown.config.ts
  • pnpm install --frozen-lockfile --ignore-scripts --offline
  • verified generated package exports, CLI, README, LICENSE, and migration copies
  • verified removed files have no stale tracked references

Summary by cubic

Removes obsolete repository files and simplifies the build configuration.

  • Removes the local PayKit architecture skill, .nvmrc, contributing guide, and release-backfill notes.
  • Pins Node 22 directly in release.yml and verify-changesets.yml instead of reading .nvmrc.
  • Inlines the PayKit tsdown config into packages/paykit/tsdown.config.ts and deletes tsdown.base.ts.
  • Removes the unused lint-staged dependency and its lockfile entries.
  • Removes the CONTRIBUTING link from the README.

Written for commit a616257. Summary will update on new commits.

Review in cubic

Summary by CodeRabbit

  • Chores

    • Release and verification workflows now use Node.js 22 directly.
    • Removed the repository’s Node.js version file and unused lint-staged dependency.
    • Simplified package build configuration while preserving existing output behavior.
  • Documentation

    • Removed outdated contribution, release backfill, and architecture guidance documents.
    • Removed the README link to the contribution guide.

@vercel

vercel Bot commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
paykit Building Building Preview Sep 16, 2026 6:30pm UTC

@coderabbitai

coderabbitai Bot commented Sep 16, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Advanced

Run ID: b49a78dc-1b37-41ee-9191-47541cc050f9

📥 Commits

Reviewing files that changed from the base of the PR and between 041fc75 and a616257.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (12)
  • .agents/skills/paykit-architecture/SKILL.md
  • .github/release-backfill/README.md
  • .github/release-backfill/v0.0.6.md
  • .github/release-backfill/v0.1.6.md
  • .github/workflows/release.yml
  • .github/workflows/verify-changesets.yml
  • .nvmrc
  • CONTRIBUTING.md
  • README.md
  • package.json
  • packages/paykit/tsdown.config.ts
  • tsdown.base.ts

📝 Walkthrough

Walkthrough

The pull request inlines the PayKit build configuration, pins Node.js 22 in two workflows, removes .nvmrc, and deletes obsolete skills, release notes, contribution guidance, and the lint-staged dependency.

Changes

Repository maintenance

Layer / File(s) Summary
Inline PayKit build configuration
packages/paykit/tsdown.config.ts, tsdown.base.ts
The PayKit package now defines its tsdown settings inline and runs preparePackageDist after successful builds. The shared configuration helper and type are deleted.
Pin Node.js in automation
.github/workflows/release.yml, .github/workflows/verify-changesets.yml, .nvmrc
Both workflows use Node.js 22 directly. The .nvmrc file is removed.
Remove obsolete repository content
.agents/skills/..., .github/release-backfill/..., CONTRIBUTING.md, README.md, package.json
The architecture skill, release backfill documents, contribution guide, README contribution link, and lint-staged dependency are removed.

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/cleanup

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

A rabbit trims the build path clean
Node twenty-two now leads the scene
Old notes hop out of sight
Inline settings build just right
Fresh outputs greet the light

Comment @coderabbitai help to get the list of available commands.

@maxktz
maxktz merged commit e74c313 into main Sep 16, 2026
7 of 11 checks passed
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.

1 participant