Skip to content

Conversation

@DaniAkash
Copy link
Contributor

This pull request updates several GitHub Actions workflow files to standardize dependency installation and simplify configuration. The main changes involve switching from bun install to bun ci for installing dependencies, removing explicit Bun version specifications, and making minor formatting adjustments for consistency.

Dependency installation improvements:

  • Replaced bun install (and bun install --frozen-lockfile) with bun ci in code-quality.yml, release-agent-sdk.yml, and test.yml to ensure reproducible and faster installs in CI environments. [1] [2] [3]

Workflow configuration simplification:

  • Removed explicit bun-version specifications from Bun setup steps in audit.yml, release-agent-sdk.yml, and test.yml to use the default version. [1] [2] [3]
  • Updated string formatting for cron schedules and Node.js setup parameters to use double quotes for consistency. [1] [2]

@github-actions github-actions bot added the fix label Jan 22, 2026
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 22, 2026

Greptile Summary

This PR standardizes GitHub Actions workflow files by switching from bun install to bun ci for dependency installation and removing explicit Bun version specifications. The bun ci command (equivalent to bun install --frozen-lockfile) ensures reproducible builds in CI environments by strictly following the lockfile, preventing unexpected dependency updates. The PR also updates string formatting to use double quotes consistently in workflow parameters.

Changes:

  • Replaced bun install and bun install --frozen-lockfile with bun ci across 4 workflow files
  • Removed explicit bun-version specifications from setup steps, allowing workflows to use default versions
  • Standardized string quotes to double quotes for cron, node-version, and registry-url parameters

Note: One workflow file (.github/workflows/branch-cleaner.yml) still uses single quotes for cron, creating a minor inconsistency with the updated style.

Confidence Score: 5/5

  • Safe to merge with minimal risk
  • Changes are limited to CI configuration improvements with no impact on runtime behavior. The switch to bun ci is a best practice for CI environments, and removing version pinning allows workflows to use maintained versions. Only minor style inconsistency found.
  • No files require special attention. Minor style inconsistency exists with .github/workflows/branch-cleaner.yml (not in this PR)

Important Files Changed

Filename Overview
.github/workflows/audit.yml Updated dependency installation from bun install to bun ci, removed explicit bun-version, and changed cron formatting to double quotes for consistency
.github/workflows/code-quality.yml Replaced bun install with bun ci for reproducible CI installs
.github/workflows/release-agent-sdk.yml Removed explicit bun-version, changed Node.js parameters to double quotes, and replaced bun install --frozen-lockfile with bun ci
.github/workflows/test.yml Removed explicit bun-version and replaced bun install --frozen-lockfile with bun ci

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant GH as GitHub Actions
    participant Bun as Bun Package Manager
    participant Lock as bun.lockb

    Dev->>GH: Push commit / Create PR
    GH->>GH: Trigger workflow
    GH->>Bun: bun ci
    Note over Bun,Lock: Before: bun install<br/>Could update dependencies
    Bun->>Lock: Read bun.lockb
    Lock-->>Bun: Exact dependency versions
    Bun->>Bun: Install exact versions<br/>(frozen lockfile)
    Note over Bun: Ensures reproducible builds<br/>Faster installs in CI
    Bun-->>GH: Dependencies installed
    GH->>GH: Run tests/builds
    GH-->>Dev: Workflow complete
Loading

@claude
Copy link

claude bot commented Jan 22, 2026

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

@claude
Copy link

claude bot commented Jan 22, 2026


Code review

No issues found. Checked for bugs and CLAUDE.md compliance.


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants