feat: add changesets, oxfmt, renovate#1
Conversation
|
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
Ignoring alerts on:
|
There was a problem hiding this comment.
Pull Request Overview
This PR sets up tooling infrastructure for a monorepo, including changesets for version management, oxfmt for code formatting, and Renovate for automated dependency updates.
Key Changes:
- Added changesets configuration for managing releases and changelogs
- Configured oxfmt as the code formatter with custom formatting rules
- Set up Renovate bot with GitHub Actions workflows for automated dependency updates
Reviewed Changes
Copilot reviewed 9 out of 10 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| pnpm-workspace.yaml | Configures pnpm workspace with TypeScript catalog, release age policies, and trust policies for dependencies |
| package.json | Adds root package configuration with oxfmt and changesets dependencies, format and release scripts |
| .oxfmtrc.jsonc | Configures oxfmt formatter settings including print width, quotes, and ignore patterns |
| .github/workflows/renovate.yml | Workflow to automatically add changesets to Renovate dependency update PRs |
| .github/workflows/release.yml | Workflow to handle releases using changesets on main branch pushes |
| .github/workflows/format-if-needed.yml | Workflow to automatically format code when pushed to main |
| .github/renovate.json | Configures Renovate bot with extended presets and ignored rules |
| .editorconfig | Sets editor configuration for consistent formatting across IDEs |
| .changeset/config.json | Configures changesets behavior including changelog format and internal dependency handling |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -0,0 +1,16 @@ | |||
| ; editorconfig.org | |||
| root = true | |||
| charset= utf8 | |||
There was a problem hiding this comment.
Missing space before the equals sign. Should be charset = utf8 to follow standard EditorConfig formatting.
| - groq@3.88.1-typegen-experimental.0 | ||
| - reselect@5.1.1 | ||
| - rxjs@7.8.2 | ||
| - semver@5.7.2 || 6.3.1 |
There was a problem hiding this comment.
The trust policy exclusion uses an OR operator ('||') in the version specification, which is unusual syntax for this context. Verify this is valid pnpm trust policy syntax. Consider splitting into separate entries if this doesn't work as intended.
| - semver@5.7.2 || 6.3.1 | |
| - semver@5.7.2 | |
| - semver@6.3.1 |
|
@SocketSecurity ignore npm/safer-buffer@2.1.2 |
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 9 out of 10 changed files in this pull request and generated 2 comments.
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| - groq@3.88.1-typegen-experimental.0 | ||
| - reselect@5.1.1 | ||
| - rxjs@7.8.2 | ||
| - semver@5.7.2 || 6.3.1 |
There was a problem hiding this comment.
The trust policy exclude pattern uses '||' which is not standard YAML syntax for version ranges. This should likely be separate entries or use pnpm's version range syntax.
| - semver@5.7.2 || 6.3.1 | |
| - semver@5.7.2 | |
| - semver@6.3.1 |
| release: | ||
| uses: sanity-io/.github/.github/workflows/changesets.yml@main | ||
| permissions: | ||
| contents: read # for checkout |
There was a problem hiding this comment.
The release workflow requires write permissions to contents to create releases and update changelogs, but only has read permissions specified. This will prevent the workflow from completing its intended release operations.
| contents: read # for checkout | |
| contents: write # for release creation and changelog updates |
No description provided.