Skip to content

Add autofix.yml GitHub Actions workflow#50

Merged
slavingia merged 3 commits intomainfrom
devin/1755113741-add-autofix-workflow
Aug 13, 2025
Merged

Add autofix.yml GitHub Actions workflow#50
slavingia merged 3 commits intomainfrom
devin/1755113741-add-autofix-workflow

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

Add autofix.yml GitHub Actions workflow

Summary

Adds an autofix.ci workflow to automatically fix linting and formatting issues on pull requests, following the pattern established in other antiwork repos (helper, gumboard, etc.).

Key changes:

  • Created .github/workflows/autofix.yml with lint auto-fixing and typecheck steps
  • Uses npm and Node 18 to match existing test workflow
  • Leverages ESLint's --fix flag since jacquez doesn't use Prettier
  • Uses npm run build for type checking

Review & Testing Checklist for Human

  • 🚨 CRITICAL: Environment variables - The typecheck step runs npm run build but doesn't include the GitHub App secrets that the existing test.yml requires. This will likely cause CI failures. Consider either adding the required env vars or using a different typecheck approach.
  • Test the workflow - Create a test PR with linting issues to verify the autofix workflow runs successfully and actually fixes the issues
  • Verify runner availability - Confirm that ubicloud-standard-2 is available and appropriate for this repo (copied from gumboard pattern)

Diagram

%%{ init : { "theme" : "default" }}%%
graph TD
    TestYML["/.github/workflows/test.yml<br/>(existing)"]:::context
    AutofixYML["/.github/workflows/autofix.yml<br/>(new workflow)"]:::major-edit
    PackageJSON["/package.json<br/>(npm scripts)"]:::context
    EslintConfig["/.eslintrc.json<br/>(lint config)"]:::context
    
    TestYML --> PackageJSON
    AutofixYML --> PackageJSON
    AutofixYML --> EslintConfig
    
    TestYML -.->|"requires env vars<br/>for build"| BuildSecrets[GitHub App Secrets]:::context
    AutofixYML -.->|"missing env vars!"| BuildSecrets
    
    subgraph Legend
        L1[Major Edit]:::major-edit
        L2[Minor Edit]:::minor-edit
        L3[Context/No Edit]:::context
    end

classDef major-edit fill:#90EE90
classDef minor-edit fill:#87CEEB  
classDef context fill:#FFFFFF
Loading

Notes

  • Pattern adapted from gumboard's autofix.yml since both repos use npm (vs helper which uses pnpm)
  • ESLint handles formatting since no Prettier config exists in jacquez
  • The workflow will run on every PR and automatically commit fixes back to the branch
  • Environment issue: The build step likely needs the same GitHub App secrets as the test workflow, but they're not included

Link to Devin run: https://app.devin.ai/sessions/42bc81cd79f648b1aae8f6eb421aed89
Requested by: @slavingia

- Follows patterns from gumboard and helper repos
- Uses npm and Node 18 to match existing test workflow
- Includes lint with --fix for automatic code formatting
- Uses typecheck via npm run build
- Configured for autofix.ci integration

Co-Authored-By: Sahil Lavingia <sahil@gumroad.com>
@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@vercel
Copy link
Copy Markdown

vercel Bot commented Aug 13, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Project Deployment Preview Comments Updated (UTC)
jacquez Ready Preview Comment Aug 13, 2025 7:43pm

- Include GH_CLIENT_SECRET and other GitHub secrets for build step
- Matches environment setup from test.yml workflow
- Ensures typecheck step can complete successfully

Co-Authored-By: Sahil Lavingia <sahil@gumroad.com>
@slavingia slavingia merged commit 792fe72 into main Aug 13, 2025
4 checks passed
@slavingia slavingia deleted the devin/1755113741-add-autofix-workflow branch August 13, 2025 19:44
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