fix: use Context::add() to prevent tasks linking to wrong process (#20)

[r2luna]

Context::push() accumulated values into a flat array, causing tasks to
always reference the first process. Context::add() replaces the value,
ensuring tasks link to their actual parent process.

Co-Authored-By: Claude Opus 4.6 [email protected]

What:

• Bug Fix
• New Feature

Description:

Related:

Summary by CodeRabbit

• Refactor
  • Updated internal process context handling mechanism for improved data structure management.

[coderabbitai]

📝 Walkthrough

Walkthrough

This PR modifies the Process class constructor to change how process context is stored. Instead of pushing three separate values to Context, it now adds a single array payload containing the process name and UUID using the Context::add method.

Changes

Cohort / File(s)	Summary
Context Storage Update src/Process.php	Replaced Context::push with Context::add method, consolidating three separate arguments into a single array payload for process context storage.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related issues

• Bug: Context::push() causes tasks to be linked to wrong process #20: The PR directly addresses this issue by implementing the exact change proposed—replacing Context::push('process', $name, $uuid) with Context::add('process', [$name, $uuid]) to prevent Context from accumulating values.

Poem

🐰 A hop, a skip, a change so neat,
Context now stores values complete,
One array instead of three,
More efficient, you will see!
Push becomes add, oh what a treat! ✨

🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Merge Conflict Detection	⚠️ Warning	❌ Merge conflicts detected (29 files): ⚔️ .github/workflows/formats.yml (content) ⚔️ .github/workflows/tests.yml (content) ⚔️ composer.json (content) ⚔️ config/brain.php (content) ⚔️ src/BrainServiceProvider.php (content) ⚔️ src/Console/BaseCommand.php (content) ⚔️ src/Console/BrainMap.php (content) ⚔️ src/Console/Printer.php (content) ⚔️ src/Console/ShowBrainCommand.php (content) ⚔️ src/Exceptions/InvalidPayload.php (content) ⚔️ src/Facades/Terminal.php (content) ⚔️ src/Process.php (content) ⚔️ src/Processes/Events/BaseEvent.php (content) ⚔️ src/Processes/Events/Error.php (content) ⚔️ src/Processes/Events/Processed.php (content) ⚔️ src/Processes/Events/Processing.php (content) ⚔️ src/Processes/Listeners/LogEventListener.php (content) ⚔️ src/Task.php (content) ⚔️ src/Tasks/Events/BaseEvent.php (content) ⚔️ src/Tasks/Events/Cancelled.php (content) ⚔️ src/Tasks/Events/Error.php (content) ⚔️ src/Tasks/Events/Processed.php (content) ⚔️ src/Tasks/Events/Processing.php (content) ⚔️ src/Tasks/Events/Skipped.php (content) ⚔️ src/Tasks/Listeners/LogEventListener.php (content) ⚔️ src/Tasks/Middleware/FinalizeTaskMiddleware.php (content) ⚔️ src/Tests/Console/MakeTestCommand.php (content) ⚔️ tests/Feature/Console/BrainMapTest.php (content) ⚔️ tests/Feature/Console/PrinterTest.php (content) These conflicts must be resolved before merging into main.	Resolve conflicts locally and push changes to this branch.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: switching from Context::push() to Context::add() to fix task-to-process linking issues.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

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

✨ 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 fix/context-process

⚔️ Resolve merge conflicts (beta)

• Auto-commit resolved conflicts to branch fix/context-process
• Create stacked PR with resolved conflicts
• Post resolved changes as copyable diffs in a comment

---

No actionable comments were generated in the recent review. 🎉

---

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