feat(spec): add containerName field to DAG-level container

[yottahmd]

No description provided.

[coderabbitai]

Warning

Rate limit exceeded

@yottahmd has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 11 minutes and 11 seconds before requesting another review.

⌛ 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.

📥 Commits

Reviewing files that changed from the base of the PR and between f04786c and 3d0e9dc.

📒 Files selected for processing (8)

• internal/core/container.go (1 hunks)
• internal/core/spec/builder.go (1 hunks)
• internal/core/spec/builder_test.go (1 hunks)
• internal/core/spec/definition.go (1 hunks)
• internal/runtime/builtin/docker/client.go (1 hunks)
• internal/runtime/builtin/docker/client_test.go (2 hunks)
• internal/runtime/builtin/docker/config.go (1 hunks)
• schemas/dag.schema.json (1 hunks)

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch containerName

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[codecov]

Codecov Report

❌ Patch coverage is 65.21739% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 59.89%. Comparing base (f04786c) to head (3d0e9dc).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
internal/runtime/builtin/docker/client.go	0.00%	8 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1496      +/-   ##
==========================================
- Coverage   59.91%   59.89%   -0.03%     
==========================================
  Files         196      196              
  Lines       22013    22023      +10     
==========================================
  Hits        13190    13190              
- Misses       7409     7418       +9     
- Partials     1414     1415       +1     

Files with missing lines	Coverage Δ
internal/core/container.go	55.55% <ø> (ø)
internal/core/spec/builder.go	68.41% <100.00%> (+0.02%)	⬆️
internal/runtime/builtin/docker/config.go	90.62% <100.00%> (+0.07%)	⬆️
internal/runtime/builtin/docker/client.go	0.00% <0.00%> (ø)

... and 1 file with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f04786c...3d0e9dc. Read the comment docs.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.