Update Github Actions

[NolanTrem]

🚀	This description was created by Ellipsis for commit e4b5d87

Summary:

This pull request updates GitHub Actions workflows, switches API version to v2, and modifies Docker configurations across multiple files.

Key points:

• Updated .github/workflows/build-main.yml to use new Docker credentials and paths.
• Removed .github/workflows/build-mainv2.yml, .github/workflows/build-release.yml, .github/workflows/integration-test-workflow-windows.yml, and .github/workflows/links.yml.
• Updated .github/workflows/integration-test-workflow-debian.yml to use ./py/Dockerfile and set working directory for tests.
• Renamed .github/workflows/ci.yml to .github/workflows/py-ci-cd.yml and updated runner to ubuntu-latest.
• Updated .github/workflows/r2r-js-sdk-ci.yml and .github/workflows/r2r-js-sdk-integration-tests.yml to use new paths and health check endpoint.
• Changed API version from v1 to v2 in go/cli/rag_operations.go, go/cli/server_operations.go, go/sdk/internal/tests/integration/management_test.go, js/sdk/__tests__/r2rClient.test.ts, and js/sdk/src/r2rClient.ts.
• Updated Docker image and health check in py/compose.yaml.
• Removed unused @click.pass_context in py/cli/commands/server.py.

---

Generated with ❤️ by ellipsis.dev

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to e4b5d87 in 51 seconds

More details

• Looked at 687 lines of code in 18 files
• Skipped 0 files when reviewing.
• Skipped posting 7 drafted comments based on config settings.

1. .github/workflows/build-main.yml:80

• Draft comment:
  Ensure that the path ./py/Dockerfile.unstructured is correct and that the Dockerfile exists at this location. This change is also made in integration-test-workflow-debian.yml and should be verified there as well.
• Reason this comment was not posted:
  Decided after close inspection that this draft comment was likely wrong and/or not actionable:
  The comment violates the rule against asking for verification. It also references a file not in the current diff, which is against the rules. The path change is something that would be caught by the build if incorrect.
  I might be overlooking the importance of verifying paths in CI/CD workflows, but the rules clearly state not to ask for verification.
  The rules are clear about not asking for verification and not commenting on files not in the diff. The build process should catch any path issues.
  Remove the comment as it violates the rules by asking for verification and referencing a file not in the diff.

2. .github/workflows/integration-test-workflow-debian.yml:55

• Draft comment:
  Ensure that the path ./py/Dockerfile is correct and that the Dockerfile exists at this location. This change is also made in build-main.yml and should be verified there as well.
• Reason this comment was not posted:
  Marked as duplicate.

3. js/sdk/__tests__/r2rClient.test.ts:14

• Draft comment:
  Ensure that the base URL http://0.0.0.0:8000/v2 is correct and consistent with the server configuration. This change is also made in src/r2rClient.ts and should be verified there as well.
• Reason this comment was not posted:
  Decided after close inspection that this draft comment was likely wrong and/or not actionable:
  The comment violates several rules: it asks for verification of the base URL, which is not necessary as it would be caught by tests or the build process. It also references a file not included in the diff, which is against the rules. The change is straightforward and does not require additional verification.
  I might be overlooking the importance of ensuring consistency between test and implementation files, but this is outside the scope of the current diff review.
  The rules clearly state not to comment on files not included in the diff, and consistency checks are typically handled by tests or during code review, not by automated comments.
  The comment should be removed as it violates the rules by asking for verification and referencing a file not in the diff.

4. js/sdk/src/r2rClient.ts:60

• Draft comment:
  Ensure that the base URL http://0.0.0.0:8000/v2 is correct and consistent with the server configuration. This change is also made in __tests__/r2rClient.test.ts and should be verified there as well.
• Reason this comment was not posted:
  Marked as duplicate.

5. py/compose.yaml:89

• Draft comment:
  Ensure that the health check URL http://localhost:${PORT:-8000}/v2/health is correct and consistent with the server configuration. This change is also made in other files and should be verified there as well.
• Reason this comment was not posted:
  Marked as duplicate.

6. go/cli/rag_operations.go:25

• Draft comment:
  Ensure that the base URL http://localhost:8000/v2 is correct and consistent with the server configuration. This change is also made in server_operations.go and should be verified there as well.
• Reason this comment was not posted:
  Marked as duplicate.

7. go/cli/server_operations.go:78

• Draft comment:
  Ensure that the base URL http://localhost:8000/v2 is correct and consistent with the server configuration. This change is also made in rag_operations.go and should be verified there as well.
• Reason this comment was not posted:
  Marked as duplicate.

Workflow ID: wflow_yBd4piphNW8XKqs0

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.