Version 0.56.0 #1938
Open
Version 0.56.0 #1938
+73,704
−39,228
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Bumps [actions/checkout](https://github.com/actions/checkout) from 5 to 6. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v5...v6) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: '6' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [vue](https://github.com/vuejs/core) from 3.5.24 to 3.5.25. - [Release notes](https://github.com/vuejs/core/releases) - [Changelog](https://github.com/vuejs/core/blob/main/CHANGELOG.md) - [Commits](vuejs/core@v3.5.24...v3.5.25) --- updated-dependencies: - dependency-name: vue dependency-version: 3.5.25 dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps the open-telemetry group in /backend with 1 update: [@opentelemetry/instrumentation-pg](https://github.com/open-telemetry/opentelemetry-js-contrib/tree/HEAD/packages/instrumentation-pg). Updates `@opentelemetry/instrumentation-pg` from 0.61.0 to 0.61.1 - [Release notes](https://github.com/open-telemetry/opentelemetry-js-contrib/releases) - [Changelog](https://github.com/open-telemetry/opentelemetry-js-contrib/blob/main/packages/instrumentation-pg/CHANGELOG.md) - [Commits](https://github.com/open-telemetry/opentelemetry-js-contrib/commits/instrumentation-pg-v0.61.1/packages/instrumentation-pg) --- updated-dependencies: - dependency-name: "@opentelemetry/instrumentation-pg" dependency-version: 0.61.1 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: open-telemetry ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [ts-loader](https://github.com/TypeStrong/ts-loader) from 9.5.2 to 9.5.4. - [Release notes](https://github.com/TypeStrong/ts-loader/releases) - [Changelog](https://github.com/TypeStrong/ts-loader/blob/main/CHANGELOG.md) - [Commits](TypeStrong/ts-loader@v9.5.2...v9.5.4) --- updated-dependencies: - dependency-name: ts-loader dependency-version: 9.5.4 dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [vue](https://github.com/vuejs/core) from 3.5.24 to 3.5.25. - [Release notes](https://github.com/vuejs/core/releases) - [Changelog](https://github.com/vuejs/core/blob/main/CHANGELOG.md) - [Commits](vuejs/core@v3.5.24...v3.5.25) --- updated-dependencies: - dependency-name: vue dependency-version: 3.5.25 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [vue-tsc](https://github.com/vuejs/language-tools/tree/HEAD/packages/tsc) from 3.1.1 to 3.1.5. - [Release notes](https://github.com/vuejs/language-tools/releases) - [Changelog](https://github.com/vuejs/language-tools/blob/master/CHANGELOG.md) - [Commits](https://github.com/vuejs/language-tools/commits/v3.1.5/packages/tsc) --- updated-dependencies: - dependency-name: vue-tsc dependency-version: 3.1.5 dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [class-validator](https://github.com/typestack/class-validator) from 0.14.2 to 0.14.3. - [Release notes](https://github.com/typestack/class-validator/releases) - [Changelog](https://github.com/typestack/class-validator/blob/develop/CHANGELOG.md) - [Commits](typestack/class-validator@v0.14.2...v0.14.3) --- updated-dependencies: - dependency-name: class-validator dependency-version: 0.14.3 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [@aws-sdk/client-sts](https://github.com/aws/aws-sdk-js-v3/tree/HEAD/clients/client-sts) from 3.726.1 to 3.939.0. - [Release notes](https://github.com/aws/aws-sdk-js-v3/releases) - [Changelog](https://github.com/aws/aws-sdk-js-v3/blob/main/clients/client-sts/CHANGELOG.md) - [Commits](https://github.com/aws/aws-sdk-js-v3/commits/v3.939.0/clients/client-sts) --- updated-dependencies: - dependency-name: "@aws-sdk/client-sts" dependency-version: 3.939.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [@types/passport-github](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/passport-github) from 1.1.12 to 1.1.13. - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/passport-github) --- updated-dependencies: - dependency-name: "@types/passport-github" dependency-version: 1.1.13 dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps the open-telemetry group in /queueConsumer with 1 update: [@opentelemetry/instrumentation-pg](https://github.com/open-telemetry/opentelemetry-js-contrib/tree/HEAD/packages/instrumentation-pg). Updates `@opentelemetry/instrumentation-pg` from 0.61.0 to 0.61.1 - [Release notes](https://github.com/open-telemetry/opentelemetry-js-contrib/releases) - [Changelog](https://github.com/open-telemetry/opentelemetry-js-contrib/blob/main/packages/instrumentation-pg/CHANGELOG.md) - [Commits](https://github.com/open-telemetry/opentelemetry-js-contrib/commits/instrumentation-pg-v0.61.1/packages/instrumentation-pg) --- updated-dependencies: - dependency-name: "@opentelemetry/instrumentation-pg" dependency-version: 0.61.1 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: open-telemetry ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [@types/dockerode](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/dockerode) from 3.3.46 to 3.3.47. - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/dockerode) --- updated-dependencies: - dependency-name: "@types/dockerode" dependency-version: 3.3.47 dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [@nestjs/cli](https://github.com/nestjs/nest-cli) from 11.0.10 to 11.0.12. - [Release notes](https://github.com/nestjs/nest-cli/releases) - [Changelog](https://github.com/nestjs/nest-cli/blob/master/.release-it.json) - [Commits](nestjs/nest-cli@11.0.10...11.0.12) --- updated-dependencies: - dependency-name: "@nestjs/cli" dependency-version: 11.0.12 dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
…sumer/staging/nestjs/cli-11.0.12' into staging
…sumer/staging/types/dockerode-3.3.47' into staging
…staging/types/passport-github-1.1.13' into staging
…sumer/staging/open-telemetry-3bf4e45efe' into staging
…staging/aws-sdk/client-sts-3.939.0' into staging
…taging/class-validator-0.14.3' into staging
…/staging/vue-tsc-3.1.5' into staging
…roduce initial migration, and update documentation, removing old migration files.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 232 out of 831 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| assert run_cmd(f"{CLI} list files -p {project.name} -m {mission.name}") == 0 | ||
| assert run_cmd(f"{CLI} list files") == 0 | ||
| assert run_cmd(f"{CLI} list files -p {mission.name}") == 0 | ||
| assert run_cmd(f"{CLI} list files -p {project.name}") == 0 |
There was a problem hiding this comment.
The test assertion was incorrectly changed. Line 70 should still test with mission.name as the project parameter (the original test case), not project.name. The original line tested that listing files with -p {mission.name} works, which is a valid edge case. Changing it to use project.name makes this test redundant with line 68.
Suggested change
| assert run_cmd(f"{CLI} list files -p {project.name}") == 0 | |
| assert run_cmd(f"{CLI} list files -p {mission.name}") == 0 |
| params[TAKE] = page_size | ||
| params[SKIP] = 0 | ||
| params[EXACT_MATCH] = str(exact_match).lower() # pass string rather than bool | ||
| if exact_match: |
There was a problem hiding this comment.
The exact_match parameter is only converted and added to params if it's truthy, but this means exact_match=False would not be passed to the API. Consider checking if exact_match is not None: instead to properly handle explicit False values.
Suggested change
| if exact_match: | |
| if exact_match is not None: |
| resp.raise_for_status() | ||
|
|
||
| return resp.text | ||
| return resp.json()["url"] |
There was a problem hiding this comment.
This change from resp.text to resp.json()[\"url\"] modifies the return type and structure. Ensure all callers of _get_file_download() are updated to handle the new return format, as they may expect a plain string URL rather than extracting it from a JSON response.
| 'user', | ||
| ); | ||
| creator = creatorData.user; | ||
| creator = creatorData.user; |
There was a problem hiding this comment.
The assignment creator = creatorData.user; is duplicated on line 40. Line 39 already assigns this value, making line 40 redundant. This pattern is repeated for user, externalUser, and admin as well (lines 44-45, 52-53, 60-61).
This was referenced Dec 10, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Version 0.56.0
Open Tasks