Revert "Remove the dependency on git ls-files from cli/build.rs (#3568)

[bfops]

Description of Changes

This reverts commit 53f692d (#3568)

That PR broke the build if any of the directories used as templates are not clean. It spammed lots of errors like:

error: `C:\Users\boppy\clockwork\SpacetimeDB\crates\cli/.templates/parent_parent_sdks_csharp_examples~_quickstart-chat_client/obj~/Debug/net8.0/client.pdb` wasn't a utf-8 file
 --> C:\Users\boppy\clockwork\SpacetimeDB\crates\cli/.templates/parent_parent_sdks_csharp_examples~_quickstart-chat_client/obj~/Debug/net8.0/client.pdb:0:49
  |

error: `C:\Users\boppy\clockwork\SpacetimeDB\crates\cli/.templates/parent_parent_sdks_csharp_examples~_quickstart-chat_client/obj~/Debug/net8.0/ref/client.dll` wasn't a utf-8 file
 --> C:\Users\boppy\clockwork\SpacetimeDB\crates\cli/.templates/parent_parent_sdks_csharp_examples~_quickstart-chat_client/obj~/Debug/net8.0/ref/client.dll:0:3
  |

error: `C:\Users\boppy\clockwork\SpacetimeDB\crates\cli/.templates/parent_parent_sdks_csharp_examples~_quickstart-chat_client/obj~/Debug/net8.0/refint/client.dll` wasn't a utf-8 file
 --> C:\Users\boppy\clockwork\SpacetimeDB\crates\cli/.templates/parent_parent_sdks_csharp_examples~_quickstart-chat_client/obj~/Debug/net8.0/refint/client.dll:0:3

API and ABI breaking changes

None.

Expected complexity level and risk

1

Testing

• cargo build passes on my repo now, which it wasn't before this commit

[gefjon]

In https://github.com/clockworklabs/SpacetimeDB/actions/runs/19641259888/job/56245061387 , I'm seeing a dirty worktree in CI. This is concerning to me; how can the worktree not be clean in our CI runner?

I would also like to know if we see any alternative path to reducing or removing our usage of git in build scripts, given the problems it's caused both in Git builds and among our users.

[jdetter]

I can confirm that I hit the same flood of errors on windows, @drogus could you give this another shot when you get a chance?

[bfops]

In https://github.com/clockworklabs/SpacetimeDB/actions/runs/19641259888/job/56245061387 , I'm seeing a dirty worktree in CI. This is concerning to me; how can the worktree not be clean in our CI runner?

Because the CI had the audacity to build and test the quickstart-chat module before trying to build the CLI.

I would also like to know if we see any alternative path to reducing or removing our usage of git in build scripts, given the problems it's caused both in Git builds and among our users.

Agreed. Created Remove git dependency in CLI build.rs.