Skip to content

feat: add event loop for async array callbacks (Promise.all, .map, .forEach)#35

Merged
TheUncharted merged 1 commit intomasterfrom
develop
Mar 13, 2026
Merged

feat: add event loop for async array callbacks (Promise.all, .map, .forEach)#35
TheUncharted merged 1 commit intomasterfrom
develop

Conversation

@TheUncharted
Copy link
Copy Markdown
Owner

@TheUncharted TheUncharted commented Mar 13, 2026

Summary by CodeRabbit

  • Documentation
    • Updated platform headline to clarify terminology, specifying "AI-generated code" in place of "AI code" to provide a clearer description of the platform's capabilities.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 13, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: f667b5fa-5307-4b13-abcf-0bd1347f7854

📥 Commits

Reviewing files that changed from the base of the PR and between e828726 and 5e756a0.

📒 Files selected for processing (1)
  • README.md

📝 Walkthrough

Walkthrough

The README headline has been updated to include the word "generated" in the description of what Zapcode runs, changing from "Run AI code. Safely. Instantly." to "Run AI-generated code. Safely. Instantly."

Changes

Cohort / File(s) Summary
Documentation Update
README.md
Updated headline to include "generated" in the description of AI code execution capabilities.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title mentions adding an event loop for async array callbacks, but the actual change only updates the README headline text. Update the title to accurately reflect the README change, such as 'docs: update README headline description' or revert to the intended async feature implementation.
Description check ⚠️ Warning No pull request description was provided by the author, but the repository requires one following the specified template. Add a description following the template with sections for Summary, Changes, Test plan, and Related issues.
✅ Passed checks (1 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch develop
📝 Coding Plan
  • Generate coding plan for human review comments

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

@github-actions
Copy link
Copy Markdown
Contributor

Benchmark Results

    Updating crates.io index
   Compiling zapcode-core v1.4.0 (/home/runner/work/zapcode/zapcode/crates/zapcode-core)
    Finished `bench` profile [optimized] target(s) in 8.39s
     Running unittests src/lib.rs (target/release/deps/zapcode_core-b075bfc4ac855948)

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

     Running benches/execution.rs (target/release/deps/execution-b4cebed678edeb20)
Timer precision: 20 ns
execution                  fastest       │ slowest       │ median        │ mean          │ samples │ iters
├─ array_creation          3.755 µs      │ 76.24 µs      │ 3.917 µs      │ 5.066 µs      │ 100     │ 100
├─ async_map_3             13.69 µs      │ 64.27 µs      │ 14.2 µs       │ 15.27 µs      │ 100     │ 100
├─ fibonacci_10            179.3 µs      │ 240.9 µs      │ 181.3 µs      │ 185.2 µs      │ 100     │ 100
├─ function_call           6.581 µs      │ 19.89 µs      │ 6.792 µs      │ 7.171 µs      │ 100     │ 100
├─ loop_100                104.5 µs      │ 135.4 µs      │ 105.6 µs      │ 106.8 µs      │ 100     │ 100
├─ object_creation         7.493 µs      │ 26.06 µs      │ 7.713 µs      │ 8.155 µs      │ 100     │ 100
├─ promise_all_3           10.11 µs      │ 32.52 µs      │ 10.43 µs      │ 10.87 µs      │ 100     │ 100
├─ promise_catch_resolved  6.432 µs      │ 30.48 µs      │ 6.671 µs      │ 7.075 µs      │ 100     │ 100
├─ promise_resolve_await   4.557 µs      │ 16.07 µs      │ 4.708 µs      │ 4.998 µs      │ 100     │ 100
├─ promise_then_chain_3    13.69 µs      │ 44.16 µs      │ 14.39 µs      │ 14.94 µs      │ 100     │ 100
├─ promise_then_single     7.994 µs      │ 21.49 µs      │ 8.225 µs      │ 8.595 µs      │ 100     │ 100
├─ simple_expression       3.426 µs      │ 11.04 µs      │ 3.605 µs      │ 3.782 µs      │ 100     │ 100
├─ string_concat           4.066 µs      │ 13.11 µs      │ 4.247 µs      │ 4.376 µs      │ 100     │ 100
├─ template_literal        4.548 µs      │ 17.59 µs      │ 4.718 µs      │ 4.906 µs      │ 100     │ 100
╰─ variable_arithmetic     4.688 µs      │ 14.18 µs      │ 4.873 µs      │ 4.999 µs      │ 100     │ 100


@TheUncharted TheUncharted merged commit 17ae149 into master Mar 13, 2026
20 checks passed
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