fix(deps): restore toml-eslint-parser for CI deploy

[waynesun09]

Summary

• Restores toml-eslint-parser to devDependencies — it was removed in 758ce3e as "unused" but is imported by cloudflare_site/scripts/patch-wrangler-rate-limit-namespace-ids.mjs, which runs in the site-deploy workflow
• Fixes: https://github.com/fullsend-ai/fullsend/actions/runs/28042265276/job/83011287095

Test plan

• site-deploy workflow passes the "Patch wrangler.toml for CI" step

[qodo-code-review]

PR Summary by Qodo

Restore toml-eslint-parser dependency required by site-deploy CI script
🐞 Bug fix ⚙️ Configuration changes 🕐 Less than 10 minutes

Description

• Restore toml-eslint-parser devDependency required by the wrangler.toml patch script.
• Unblock the site-deploy workflow step that patches wrangler.toml during deploys.

Diagram

graph TD
  A[["site-deploy.yml"]] --> B(("Patch wrangler.toml script")) --> C["wrangler.toml"]
  B --> D["toml-eslint-parser"]
  subgraph Legend
    direction LR
    _wf[["Workflow"]] ~~~ _scr(("Script")) ~~~ _cfg["Config/Dep"]
  end

Loading

High-Level Assessment

The following are alternative approaches to this PR:

1. Move parser to runtime dependencies

• ➕ Avoids future CI breakage if tooling/scripts are overlooked during dependency pruning
• ➕ Clarifies that deploy workflows rely on this package
• ➖ Slightly expands runtime dependency surface even though it’s only used in CI/script paths

2. Replace with a simpler TOML library (e.g., @iarna/toml)

• ➕ Potentially smaller dependency surface
• ➕ More common TOML parsing API
• ➖ Likely loses source-range preservation, risking formatting/comment churn in wrangler.toml edits
• ➖ Would require script changes and additional testing

3. Vendor a minimal TOML patcher (regex/range-less)

• ➕ No external parser dependency
• ➖ Fragile against formatting changes and comments; higher maintenance burden
• ➖ Higher risk of corrupting wrangler.toml in CI

Recommendation: Keep the current approach: restoring toml-eslint-parser is the smallest, lowest-risk fix that preserves the script’s source-range editing behavior. Consider adding a brief comment or CI check to prevent dependency pruning from removing packages imported by cloudflare_site/scripts/*.

Files changed (1) +1 / -0

Other (1) +1 / -0

package.jsonRe-add toml-eslint-parser devDependency for CI wrangler.toml patching +1/-0

Re-add toml-eslint-parser devDependency for CI wrangler.toml patching

• Adds 'toml-eslint-parser' back to 'devDependencies'. This restores the dependency required by 'cloudflare_site/scripts/patch-wrangler-rate-limit-namespace-ids.mjs', which runs during the site deploy workflow.

package.json

[qodo-code-review]

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

[fullsend-ai-review]

🤖 Finished Review · ✅ Success · Started 5:12 PM UTC · Completed 5:15 PM UTC
Commit: 6fa766e · View workflow run →

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[fullsend-ai-review]

Looks good to me

---

Labels: Restores a devDependency required by the CI site-deploy workflow.