fix(build): Add missing Spider scheduler and worker binaries to task package's source list (fixes #1409).

[sitaowang1998]

Description

This PR resolves #1409 by adding spider_scheduler and spider_worker to the package task's sources list, so the package will rerun if either executable changes.

Checklist

• The PR satisfies the contribution guidelines.
• This is a breaking change and that has been indicated in the PR title, OR this isn't a
  breaking change.
• Necessary docs have been updated, OR no docs need to be updated.

Validation performed

• Modify either executable triggers package task.

Summary by CodeRabbit

• New Features

  • The package now includes Spider Scheduler and Spider Worker components, making these modules available after installation and deployment.

• Chores

  • Updated packaging to assemble and generate checksums for the new Spider components, ensuring they are included in distribution artefacts with consistent integrity verification.

[coderabbitai]

Walkthrough

Added two new build sources, spider_scheduler and spider_worker, to the package assembly in taskfile.yaml, extending package contents and checksum generation to include these Spider components.

Changes

Cohort / File(s)	Summary of Changes
Packaging: Taskfile updates taskfile.yaml	Included G_SPIDER_BUILD_DIR/spider-build/src/spider/spider_scheduler and .../spider_worker as package sources and in checksum generation to track Spider artifacts in packaging.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related issues

• Add checksum tracking for spider artifacts in package task #1409 — Aligns with adding spider_scheduler and spider_worker to package sources/checksums so package tasks rerun when Spider artifacts change.
• deps:spider task performs unnecessary rebuilds due to mariadb-connector-cpp #1412 — Matches updates to include Spider components in checksum generation to prevent unnecessary rebuilds and ensure proper invalidation.

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title clearly and succinctly describes the primary change by stating the build fix and specifying that missing Spider scheduler and worker binaries are added to the package task’s source list, while also referencing the relevant issue number.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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

[junhaoliao]

(Just to show that we've considered this alternative -) one option is to have deps:spider generate a checksum file, which the package task could then reference as a single source instead of listing individual binaries. This approach might depend on the work in #1415.

Personally, I find the current approach in this PR cleaner - explicitly adding the binaries to the source list keeps the package task relatively self-contained and easier to reason about.

For the title, how about:

fix(build): Add missing Spider scheduler and worker binaries to task `package`'s source list (fixes #1409).

[Bill-hbrhbr]

@junhaoliao What about spider_task_executor?

[sitaowang1998]

@junhaoliao What about spider_task_executor?

spider_task_executor is not used in clp right now.