Skip to content

feat: add upload workflow report checks - #50

Merged
edhor1608 merged 1 commit into
mainfrom
repo-refresh-upload-report-checks
May 2, 2026
Merged

edhor1608 merged 1 commit into
mainfrom
repo-refresh-upload-report-checks

Conversation

@edhor1608

@edhor1608 edhor1608 commented May 1, 2026 •

Copy link
Copy Markdown
Owner

Add upload workflow, API image file size, and color metadata checks to report

  • Extends report.buildReport in report.ts with three new checks: upload_workflow (warns if unspecified), api_image_file_size (warns if still-image exceeds 8 MiB for api_scheduler), and color_metadata (warns if colorspace is unknown).
  • Adds a next_action for app_direct workflow advising users to enable Instagram high-quality uploads and avoid in-app edits.
  • Documents api_scheduler and app_direct workflow modes in the README, including profile and size baselines.
  • Behavioral Change: omitting --workflow now results in an upload_workflow warn status in the report instead of a silent pass.

Macroscope summarized d93ed85.

@coderabbitai

coderabbitai Bot commented May 1, 2026 •

Copy link
Copy Markdown

Warning

Rate limit exceeded

@edhor1608 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 4 minutes and 51 seconds before requesting another review.

To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 994cef81-5444-490c-982d-ccd7ad35187c

📥 Commits

Reviewing files that changed from the base of the PR and between 9742f03 and d93ed85.

📒 Files selected for processing (4)
  • README.md
  • docs/repo_refresh_audit.md
  • src/domain/report.ts
  • tests/integration/report.integration.test.ts
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch repo-refresh-upload-report-checks
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch repo-refresh-upload-report-checks

Review rate limit: 0/5 reviews remaining, refill in 4 minutes and 51 seconds.

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

@edhor1608
edhor1608 force-pushed the repo-refresh-upload-report-checks branch from a126d31 to e1b4987 Compare May 2, 2026 19:03
@edhor1608
edhor1608 marked this pull request as ready for review May 2, 2026 19:03
@edhor1608
edhor1608 force-pushed the repo-refresh-lint-gate-cleanup branch from 0425315 to ea72cc0 Compare May 2, 2026 21:25
@edhor1608
edhor1608 force-pushed the repo-refresh-upload-report-checks branch from e1b4987 to 6ca3a20 Compare May 2, 2026 21:25
@edhor1608
edhor1608 force-pushed the repo-refresh-lint-gate-cleanup branch from ea72cc0 to 06ce726 Compare May 2, 2026 21:25
@edhor1608
edhor1608 force-pushed the repo-refresh-upload-report-checks branch from 6ca3a20 to 4dcb6ce Compare May 2, 2026 21:26
@edhor1608
edhor1608 force-pushed the repo-refresh-lint-gate-cleanup branch from 06ce726 to ea72cc0 Compare May 2, 2026 21:29
@edhor1608
edhor1608 force-pushed the repo-refresh-upload-report-checks branch from 4dcb6ce to 6ca3a20 Compare May 2, 2026 21:29
@edhor1608
edhor1608 force-pushed the repo-refresh-lint-gate-cleanup branch from ea72cc0 to a5252e9 Compare May 2, 2026 21:32
@edhor1608
edhor1608 force-pushed the repo-refresh-upload-report-checks branch from 6ca3a20 to 095b5ff Compare May 2, 2026 21:32
@graphite-app
graphite-app Bot changed the base branch from repo-refresh-lint-gate-cleanup to graphite-base/50 May 2, 2026 21:38
@edhor1608
edhor1608 force-pushed the graphite-base/50 branch from a5252e9 to 9742f03 Compare May 2, 2026 21:40
@edhor1608
edhor1608 force-pushed the repo-refresh-upload-report-checks branch from 095b5ff to e54c1de Compare May 2, 2026 21:40
@edhor1608
edhor1608 force-pushed the repo-refresh-upload-report-checks branch from e54c1de to d93ed85 Compare May 2, 2026 21:41
@edhor1608
edhor1608 changed the base branch from graphite-base/50 to main May 2, 2026 21:41
@graphite-app

graphite-app Bot commented May 2, 2026

Copy link
Copy Markdown

Merge activity

  • May 2, 9:41 PM UTC: Graphite rebased this pull request, because this pull request is set to merge when ready.

@edhor1608
edhor1608 merged commit 3038201 into main May 2, 2026
3 checks passed
@edhor1608
edhor1608 deleted the repo-refresh-upload-report-checks branch May 15, 2026 18:54
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