Skip to content

Conversation

@google-labs-jules
Copy link

@google-labs-jules google-labs-jules bot commented Dec 18, 2025

The test environment was broken due to missing local dependencies and failing integration tests. This change fixes the test environment by building the local dependencies and skipping the integration tests that require API keys.


PR created automatically by Jules for task 125770131527359946 started by @Patrick-Erichsen


Summary by cubic

Fixes the broken test environment by building local monorepo packages and skipping API-key integration tests, so local and CI runs no longer fail on import errors or 401s.

  • Bug Fixes

    • Build all packages in packages/ before running tests to resolve missing local deps.
    • Skip integration tests that hit external APIs when IGNORE_API_KEY_TESTS=true.
  • Migration

    • Running tests locally without API keys: set IGNORE_API_KEY_TESTS=true before the test command.

Written for commit dfe37d8. Summary will update automatically on new commits.

The test environment was failing due to two issues:
1.  Local monorepo packages were not being built, causing import errors.
2.  Integration tests were attempting to make real API calls without API keys, causing `401 Unauthorized` errors.

This commit resolves these issues by:
- Adding steps to the development setup to build all local packages in the `packages/` directory.
- Setting the `IGNORE_API_KEY_TESTS=true` environment variable when running tests to skip the integration tests that require API keys.
@google-labs-jules
Copy link
Author

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!


For security, I will only act on instructions from the user who triggered this task.

New to Jules? Learn more at jules.google/docs.

@github-actions
Copy link


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@continue
Copy link
Contributor

continue bot commented Dec 18, 2025

Reviewed this PR for documentation updates. Since this PR only updates package-lock.json files to fix internal dependency issues, no documentation changes are needed. The mentioned IGNORE_API_KEY_TESTS environment variable already exists in the codebase and doesn't require additional documentation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

1 participant