mirror of
https://github.com/Comfy-Org/ComfyUI_frontend.git
synced 2026-04-20 14:30:41 +00:00
Compare commits
33 Commits
manager/me
...
sno-lint-i
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
2e6bf4896e | ||
|
|
b87a251e6b | ||
|
|
1aa2a66b01 | ||
|
|
253853be6f | ||
|
|
7f86f3fec1 | ||
|
|
9a381415e5 | ||
|
|
b61ecef7fd | ||
|
|
aab6c0c954 | ||
|
|
8afc0b9012 | ||
|
|
3a869d98da | ||
|
|
3021b60dc4 | ||
|
|
1643baf761 | ||
|
|
8d4846770f | ||
|
|
a88d46bdb7 | ||
|
|
39686ef0f6 | ||
|
|
943359e559 | ||
|
|
a7dd696cbe | ||
|
|
405b97c5ef | ||
|
|
e27b44bdac | ||
|
|
01a0ebc8c8 | ||
|
|
cd970bd29d | ||
|
|
ad7d23177e | ||
|
|
ccaeab1541 | ||
|
|
3024dcc41b | ||
|
|
c5581c9393 | ||
|
|
fe0054d9dd | ||
|
|
b4bf72b2b7 | ||
|
|
45b43aa093 | ||
|
|
ec2888a116 | ||
|
|
0f747f3c5d | ||
|
|
7ee33e2892 | ||
|
|
73f71fc018 | ||
|
|
6b085ea8c4 |
206
.github/workflows/auto-fix-and-update.yaml
vendored
Normal file
206
.github/workflows/auto-fix-and-update.yaml
vendored
Normal file
@@ -0,0 +1,206 @@
|
||||
name: Auto-fix and Update
|
||||
|
||||
on:
|
||||
pull_request:
|
||||
branches-ignore: [wip/*, draft/*, temp/*]
|
||||
|
||||
permissions:
|
||||
contents: write
|
||||
pull-requests: write
|
||||
issues: write
|
||||
|
||||
jobs:
|
||||
auto-fix-and-update:
|
||||
# Only run on PRs from the same repository (not forks) to avoid permission issues, auto fix any lint and formats, update locales, and commit the changes back
|
||||
if: github.event.pull_request.head.repo.full_name == github.repository
|
||||
runs-on: ubuntu-latest
|
||||
steps:
|
||||
- name: Setup ComfyUI Frontend and Backend
|
||||
uses: Comfy-Org/ComfyUI_frontend_setup_action@v2.3
|
||||
|
||||
# Step 1: Cache ESLint for better performance
|
||||
- name: Cache ESLint
|
||||
uses: actions/cache@v4
|
||||
with:
|
||||
path: ComfyUI_frontend/.eslintcache
|
||||
key: ${{ runner.os }}-eslint-${{ hashFiles('ComfyUI_frontend/eslint.config.js', 'ComfyUI_frontend/package-lock.json') }}
|
||||
restore-keys: |
|
||||
${{ runner.os }}-eslint-
|
||||
|
||||
# Step 2: Run lint and format fixes
|
||||
- name: Run ESLint with auto-fix
|
||||
run: npm run lint:fix
|
||||
working-directory: ComfyUI_frontend
|
||||
|
||||
- name: Run Prettier with auto-format
|
||||
run: npm run format
|
||||
working-directory: ComfyUI_frontend
|
||||
|
||||
# Step 3: Update locales (only if there are relevant changes)
|
||||
- name: Check if locale updates needed
|
||||
id: check-locale-changes
|
||||
run: |
|
||||
# Check if there are changes to files that would affect locales
|
||||
if git diff --name-only origin/${{ github.base_ref }}..HEAD | grep -E '\.(vue|ts|tsx)$' | grep -v test | head -1; then
|
||||
echo "locale_updates_needed=true" >> $GITHUB_OUTPUT
|
||||
else
|
||||
echo "locale_updates_needed=false" >> $GITHUB_OUTPUT
|
||||
fi
|
||||
working-directory: ComfyUI_frontend
|
||||
|
||||
- name: Install Playwright Browsers
|
||||
if: steps.check-locale-changes.outputs.locale_updates_needed == 'true'
|
||||
run: npx playwright install chromium --with-deps
|
||||
working-directory: ComfyUI_frontend
|
||||
|
||||
- name: Start dev server
|
||||
if: steps.check-locale-changes.outputs.locale_updates_needed == 'true'
|
||||
run: npm run dev:electron &
|
||||
working-directory: ComfyUI_frontend
|
||||
|
||||
- name: Update en.json
|
||||
if: steps.check-locale-changes.outputs.locale_updates_needed == 'true'
|
||||
run: npm run collect-i18n -- scripts/collect-i18n-general.ts
|
||||
env:
|
||||
PLAYWRIGHT_TEST_URL: http://localhost:5173
|
||||
working-directory: ComfyUI_frontend
|
||||
|
||||
- name: Update translations
|
||||
if: steps.check-locale-changes.outputs.locale_updates_needed == 'true'
|
||||
run: npm run locale
|
||||
env:
|
||||
OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
|
||||
working-directory: ComfyUI_frontend
|
||||
|
||||
# Step 4: Check for any changes from both lint-format and locale updates
|
||||
- name: Check for changes
|
||||
id: verify-changed-files
|
||||
run: |
|
||||
if [ -n "$(git status --porcelain)" ]; then
|
||||
echo "changed=true" >> $GITHUB_OUTPUT
|
||||
# Show what changed for debugging
|
||||
echo "Changed files:"
|
||||
git status --porcelain
|
||||
else
|
||||
echo "changed=false" >> $GITHUB_OUTPUT
|
||||
fi
|
||||
working-directory: ComfyUI_frontend
|
||||
|
||||
# Step 5: Commit all changes in a single commit
|
||||
- name: Commit auto-fixes and locale updates
|
||||
if: steps.verify-changed-files.outputs.changed == 'true'
|
||||
run: |
|
||||
git config --global user.name 'github-actions'
|
||||
git config --global user.email 'github-actions@github.com'
|
||||
git fetch origin ${{ github.head_ref }}
|
||||
# Stash any local changes before checkout
|
||||
git stash -u
|
||||
git checkout -B ${{ github.head_ref }} origin/${{ github.head_ref }}
|
||||
# Apply the stashed changes if any
|
||||
git stash pop || true
|
||||
git add .
|
||||
git diff --staged --quiet || git commit -m "[auto-fix] Apply ESLint, Prettier fixes and update locales"
|
||||
git push origin HEAD:${{ github.head_ref }}
|
||||
working-directory: ComfyUI_frontend
|
||||
|
||||
# Step 6: Run final validation
|
||||
- name: Final validation
|
||||
run: |
|
||||
npm run lint
|
||||
npm run format:check
|
||||
npm run knip
|
||||
working-directory: ComfyUI_frontend
|
||||
|
||||
# Step 7: Comment on PR about what was fixed
|
||||
- name: Comment on PR about auto-fixes
|
||||
if: steps.verify-changed-files.outputs.changed == 'true'
|
||||
uses: actions/github-script@v7
|
||||
with:
|
||||
script: |
|
||||
let changes = [];
|
||||
if ('${{ steps.check-locale-changes.outputs.locale_updates_needed }}' === 'true') {
|
||||
changes.push('Locale updates');
|
||||
}
|
||||
// Always include lint/format as we ran them
|
||||
changes.push('ESLint auto-fixes', 'Prettier formatting');
|
||||
|
||||
const changesText = changes.join('\n- ');
|
||||
|
||||
github.rest.issues.createComment({
|
||||
issue_number: context.issue.number,
|
||||
owner: context.repo.owner,
|
||||
repo: context.repo.repo,
|
||||
body: `## 🔧 Auto-fixes Applied
|
||||
|
||||
This PR has been automatically updated with the following changes:
|
||||
|
||||
- ${changesText}
|
||||
|
||||
**⚠️ Important**: Your local branch is now behind. Run \`git pull\` before making additional changes to avoid conflicts.`
|
||||
});
|
||||
|
||||
# Separate job for fork PRs that can't auto-commit, we only check lint and formats, and do not commit back
|
||||
fork-pr-check:
|
||||
if: github.event.pull_request.head.repo.full_name != github.repository
|
||||
runs-on: ubuntu-latest
|
||||
steps:
|
||||
- name: Checkout PR
|
||||
uses: actions/checkout@v4
|
||||
with:
|
||||
ref: ${{ github.event.pull_request.head.sha }}
|
||||
|
||||
- name: Use Node.js
|
||||
uses: actions/setup-node@v4
|
||||
with:
|
||||
node-version: 'lts/*'
|
||||
cache: 'npm'
|
||||
|
||||
- name: Install dependencies
|
||||
run: npm ci
|
||||
|
||||
- name: Cache ESLint
|
||||
uses: actions/cache@v4
|
||||
with:
|
||||
path: .eslintcache
|
||||
key: ${{ runner.os }}-eslint-fork-${{ hashFiles('eslint.config.js', 'package-lock.json') }}
|
||||
restore-keys: |
|
||||
${{ runner.os }}-eslint-fork-
|
||||
|
||||
- name: Check linting and formatting
|
||||
id: check-lint-format
|
||||
run: |
|
||||
# Run checks and capture exit codes
|
||||
npm run lint || echo "lint_failed=true" >> $GITHUB_OUTPUT
|
||||
npm run format:check || echo "format_failed=true" >> $GITHUB_OUTPUT
|
||||
npm run knip || echo "knip_failed=true" >> $GITHUB_OUTPUT
|
||||
|
||||
- name: Comment on fork PR about manual fixes needed
|
||||
if: steps.check-lint-format.outputs.lint_failed == 'true' || steps.check-lint-format.outputs.format_failed == 'true' || steps.check-lint-format.outputs.knip_failed == 'true'
|
||||
uses: actions/github-script@v7
|
||||
with:
|
||||
script: |
|
||||
github.rest.issues.createComment({
|
||||
issue_number: context.issue.number,
|
||||
owner: context.repo.owner,
|
||||
repo: context.repo.repo,
|
||||
body: `## ⚠️ Linting/Formatting Issues Found
|
||||
|
||||
This PR has linting or formatting issues that need to be fixed.
|
||||
|
||||
**Since this PR is from a fork, auto-fix cannot be applied automatically.**
|
||||
|
||||
### Option 1: Set up pre-commit hooks (recommended)
|
||||
Run this once to automatically format code on every commit:
|
||||
\`\`\`bash
|
||||
npm run prepare
|
||||
\`\`\`
|
||||
|
||||
### Option 2: Fix manually
|
||||
Run these commands and push the changes:
|
||||
\`\`\`bash
|
||||
npm run lint:fix
|
||||
npm run format
|
||||
\`\`\`
|
||||
|
||||
See [CONTRIBUTING.md](https://github.com/Comfy-Org/ComfyUI_frontend/blob/main/CONTRIBUTING.md#git-pre-commit-hooks) for more details.`
|
||||
});
|
||||
51
.github/workflows/i18n.yaml
vendored
51
.github/workflows/i18n.yaml
vendored
@@ -1,51 +0,0 @@
|
||||
name: Update Locales
|
||||
|
||||
on:
|
||||
pull_request:
|
||||
branches: [ main, master, dev* ]
|
||||
paths-ignore:
|
||||
- '.github/**'
|
||||
- '.husky/**'
|
||||
- '.vscode/**'
|
||||
- 'browser_tests/**'
|
||||
- 'tests-ui/**'
|
||||
|
||||
jobs:
|
||||
update-locales:
|
||||
# Don't run on fork PRs
|
||||
if: github.event.pull_request.head.repo.full_name == github.repository
|
||||
runs-on: ubuntu-latest
|
||||
steps:
|
||||
- uses: Comfy-Org/ComfyUI_frontend_setup_action@v2.3
|
||||
- name: Install Playwright Browsers
|
||||
run: npx playwright install chromium --with-deps
|
||||
working-directory: ComfyUI_frontend
|
||||
- name: Start dev server
|
||||
# Run electron dev server as it is a superset of the web dev server
|
||||
# We do want electron specific UIs to be translated.
|
||||
run: npm run dev:electron &
|
||||
working-directory: ComfyUI_frontend
|
||||
- name: Update en.json
|
||||
run: npm run collect-i18n -- scripts/collect-i18n-general.ts
|
||||
env:
|
||||
PLAYWRIGHT_TEST_URL: http://localhost:5173
|
||||
working-directory: ComfyUI_frontend
|
||||
- name: Update translations
|
||||
run: npm run locale
|
||||
env:
|
||||
OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
|
||||
working-directory: ComfyUI_frontend
|
||||
- name: Commit updated locales
|
||||
run: |
|
||||
git config --global user.name 'github-actions'
|
||||
git config --global user.email 'github-actions@github.com'
|
||||
git fetch origin ${{ github.head_ref }}
|
||||
# Stash any local changes before checkout
|
||||
git stash -u
|
||||
git checkout -B ${{ github.head_ref }} origin/${{ github.head_ref }}
|
||||
# Apply the stashed changes if any
|
||||
git stash pop || true
|
||||
git add src/locales/
|
||||
git diff --staged --quiet || git commit -m "Update locales [skip ci]"
|
||||
git push origin HEAD:${{ github.head_ref }}
|
||||
working-directory: ComfyUI_frontend
|
||||
83
.github/workflows/lint-and-format.yaml
vendored
83
.github/workflows/lint-and-format.yaml
vendored
@@ -1,83 +0,0 @@
|
||||
name: Lint and Format
|
||||
|
||||
on:
|
||||
pull_request:
|
||||
branches-ignore: [wip/*, draft/*, temp/*]
|
||||
|
||||
permissions:
|
||||
contents: write
|
||||
pull-requests: write
|
||||
|
||||
jobs:
|
||||
lint-and-format:
|
||||
runs-on: ubuntu-latest
|
||||
steps:
|
||||
- name: Checkout PR
|
||||
uses: actions/checkout@v4
|
||||
with:
|
||||
token: ${{ secrets.GITHUB_TOKEN }}
|
||||
ref: ${{ github.event.pull_request.head.sha }}
|
||||
fetch-depth: 0
|
||||
|
||||
- name: Use Node.js
|
||||
uses: actions/setup-node@v4
|
||||
with:
|
||||
node-version: 'lts/*'
|
||||
cache: 'npm'
|
||||
|
||||
- name: Install dependencies
|
||||
run: npm ci
|
||||
|
||||
- name: Run ESLint with auto-fix
|
||||
run: npm run lint:fix
|
||||
|
||||
- name: Run Prettier with auto-format
|
||||
run: npm run format
|
||||
|
||||
- name: Check for changes
|
||||
id: verify-changed-files
|
||||
run: |
|
||||
if [ -n "$(git status --porcelain)" ]; then
|
||||
echo "changed=true" >> $GITHUB_OUTPUT
|
||||
else
|
||||
echo "changed=false" >> $GITHUB_OUTPUT
|
||||
fi
|
||||
|
||||
- name: Commit changes
|
||||
if: steps.verify-changed-files.outputs.changed == 'true' && github.event.pull_request.head.repo.full_name == github.repository
|
||||
run: |
|
||||
git config --local user.email "action@github.com"
|
||||
git config --local user.name "GitHub Action"
|
||||
git add .
|
||||
git commit -m "[auto-fix] Apply ESLint and Prettier fixes"
|
||||
git push
|
||||
|
||||
- name: Final validation
|
||||
run: |
|
||||
npm run lint
|
||||
npm run format:check
|
||||
npm run knip
|
||||
|
||||
- name: Comment on PR about auto-fix
|
||||
if: steps.verify-changed-files.outputs.changed == 'true' && github.event.pull_request.head.repo.full_name == github.repository
|
||||
uses: actions/github-script@v7
|
||||
with:
|
||||
script: |
|
||||
github.rest.issues.createComment({
|
||||
issue_number: context.issue.number,
|
||||
owner: context.repo.owner,
|
||||
repo: context.repo.repo,
|
||||
body: '## 🔧 Auto-fixes Applied\n\nThis PR has been automatically updated to fix linting and formatting issues.\n\n**⚠️ Important**: Your local branch is now behind. Run `git pull` before making additional changes to avoid conflicts.\n\n### Changes made:\n- ESLint auto-fixes\n- Prettier formatting'
|
||||
})
|
||||
|
||||
- name: Comment on PR about manual fix needed
|
||||
if: steps.verify-changed-files.outputs.changed == 'true' && github.event.pull_request.head.repo.full_name != github.repository
|
||||
uses: actions/github-script@v7
|
||||
with:
|
||||
script: |
|
||||
github.rest.issues.createComment({
|
||||
issue_number: context.issue.number,
|
||||
owner: context.repo.owner,
|
||||
repo: context.repo.repo,
|
||||
body: '## ⚠️ Linting/Formatting Issues Found\n\nThis PR has linting or formatting issues that need to be fixed.\n\n**Since this PR is from a fork, auto-fix cannot be applied automatically.**\n\n### Option 1: Set up pre-commit hooks (recommended)\nRun this once to automatically format code on every commit:\n```bash\nnpm run prepare\n```\n\n### Option 2: Fix manually\nRun these commands and push the changes:\n```bash\nnpm run lint:fix\nnpm run format\n```\n\nSee [CONTRIBUTING.md](https://github.com/Comfy-Org/ComfyUI_frontend/blob/main/CONTRIBUTING.md#git-pre-commit-hooks) for more details.'
|
||||
})
|
||||
9
.github/workflows/release.yaml
vendored
9
.github/workflows/release.yaml
vendored
@@ -22,6 +22,7 @@ jobs:
|
||||
- uses: actions/setup-node@v4
|
||||
with:
|
||||
node-version: 'lts/*'
|
||||
cache: 'npm'
|
||||
- name: Get current version
|
||||
id: current_version
|
||||
run: echo "version=$(node -p "require('./package.json').version")" >> $GITHUB_OUTPUT
|
||||
@@ -34,6 +35,13 @@ jobs:
|
||||
else
|
||||
echo "is_prerelease=false" >> $GITHUB_OUTPUT
|
||||
fi
|
||||
- name: Cache Vite build
|
||||
uses: actions/cache@v4
|
||||
with:
|
||||
path: node_modules/.vite
|
||||
key: ${{ runner.os }}-vite-release-${{ hashFiles('package-lock.json') }}
|
||||
restore-keys: |
|
||||
${{ runner.os }}-vite-release-
|
||||
- name: Build project
|
||||
env:
|
||||
SENTRY_DSN: ${{ secrets.SENTRY_DSN }}
|
||||
@@ -117,6 +125,7 @@ jobs:
|
||||
with:
|
||||
node-version: 'lts/*'
|
||||
registry-url: https://registry.npmjs.org
|
||||
cache: 'npm'
|
||||
- run: npm ci
|
||||
- run: npm run build:types
|
||||
- name: Publish package
|
||||
|
||||
26
.github/workflows/test-ui.yaml
vendored
26
.github/workflows/test-ui.yaml
vendored
@@ -35,6 +35,16 @@ jobs:
|
||||
- uses: actions/setup-node@v4
|
||||
with:
|
||||
node-version: lts/*
|
||||
cache: 'npm'
|
||||
cache-dependency-path: ComfyUI_frontend/package-lock.json
|
||||
|
||||
- name: Cache Vite build
|
||||
uses: actions/cache@v4
|
||||
with:
|
||||
path: ComfyUI_frontend/node_modules/.vite
|
||||
key: ${{ runner.os }}-vite-build-${{ hashFiles('ComfyUI_frontend/package-lock.json') }}
|
||||
restore-keys: |
|
||||
${{ runner.os }}-vite-build-
|
||||
|
||||
- name: Build ComfyUI_frontend
|
||||
run: |
|
||||
@@ -92,7 +102,23 @@ jobs:
|
||||
wait-for-it --service 127.0.0.1:8188 -t 600
|
||||
working-directory: ComfyUI
|
||||
|
||||
- name: Get Playwright version
|
||||
id: playwright-version
|
||||
run: echo "version=$(npm list @playwright/test --depth=0 --json | jq -r '.dependencies["@playwright/test"].version')" >> $GITHUB_OUTPUT
|
||||
working-directory: ComfyUI_frontend
|
||||
|
||||
- name: Cache Playwright browsers
|
||||
uses: actions/cache@v4
|
||||
id: playwright-cache
|
||||
with:
|
||||
path: ~/.cache/ms-playwright
|
||||
key: ${{ runner.os }}-playwright-${{ steps.playwright-version.outputs.version }}-chromium
|
||||
restore-keys: |
|
||||
${{ runner.os }}-playwright-${{ steps.playwright-version.outputs.version }}-
|
||||
${{ runner.os }}-playwright-
|
||||
|
||||
- name: Install Playwright Browsers
|
||||
if: steps.playwright-cache.outputs.cache-hit != 'true'
|
||||
run: npx playwright install chromium --with-deps
|
||||
working-directory: ComfyUI_frontend
|
||||
|
||||
|
||||
9
.github/workflows/vitest.yaml
vendored
9
.github/workflows/vitest.yaml
vendored
@@ -17,10 +17,19 @@ jobs:
|
||||
uses: actions/setup-node@v4
|
||||
with:
|
||||
node-version: 'lts/*'
|
||||
cache: 'npm'
|
||||
|
||||
- name: Install dependencies
|
||||
run: npm ci
|
||||
|
||||
- name: Cache Vite
|
||||
uses: actions/cache@v4
|
||||
with:
|
||||
path: node_modules/.vite
|
||||
key: ${{ runner.os }}-vite-${{ hashFiles('package-lock.json') }}
|
||||
restore-keys: |
|
||||
${{ runner.os }}-vite-
|
||||
|
||||
- name: Run Vitest tests
|
||||
run: |
|
||||
npm run test:component
|
||||
|
||||
Reference in New Issue
Block a user