feat(clp-package): Warn the user if they do not use the `--timestamp-key` flag when compressing with the `clp-s` storage engine by quinntaylormitchell · Pull Request #1283 · y-scope/clp

quinntaylormitchell · 2025-09-04T13:41:30Z

Description

When using the clp-s storage engine, users are currently not warned if they do not use the --timestamp-key flag in the compression command. This PR introduces a warning that notifies the user if they do not specify the --timestamp-key flag. This will create a better experience for users whose logs have a timestamp key and who have forgotten to specify it. The warning is phrased in such a way that users whose logs do not have a timestamp key (or users who have otherwise intentionally omitted the --timestamp-key) will not be deterred from using the clp-s storage engine to compress their logs.

The warning reads as follows:

`--timestamp-key` not specified. Events will not have assigned timestamps and can only be searched from the command line without a timestamp filter.

The behaviour of compression scripts is otherwise unchanged, i.e., compression jobs will still proceed regardless of whether the user uses the --timestamp-key flag.

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

Compressed without using the --timestamp-key flag; received the warning correctly.

Summary by CodeRabbit

New Features
- Displays a runtime warning during compression after dataset validation when no timestamp key is provided, advising use of the --timestamp-key flag.
- No change to behaviour or outputs; the timestamp flag is included in the compression command only when explicitly supplied by the user.
- Does not alter public interfaces or return values.

…y flag

coderabbitai · 2025-09-04T13:41:37Z

Walkthrough

Adds a runtime warning in the CLP_S compression path: after dataset validation, if no timestamp key is provided (parsed_args.timestamp_key is None), a warning is logged suggesting use of the --timestamp-key flag. No control flow, signatures, or return values are changed.

Changes

Cohort / File(s)	Summary
CLP package utils — compress warning `components/clp-package-utils/clp_package_utils/scripts/compress.py`	Inserted a post-validation runtime warning for missing `--timestamp-key` in the CLP_S path; command construction remains unchanged and only includes the timestamp key when explicitly provided.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 Ruff (0.12.2)

components/clp-package-utils/clp_package_utils/scripts/compress.py

�[1;31mruff failed�[0m
�[1mCause:�[0m Failed to load configuration /ruff.toml
�[1mCause:�[0m Failed to parse /ruff.toml
�[1mCause:�[0m TOML parse error at line 26, column 3
|
26 | "RSE100", # Use of assert detected
| ^^^^^^^^
Unknown rule selector: RSE100

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ff1e162 and e6cb592.

📒 Files selected for processing (1)

components/clp-package-utils/clp_package_utils/scripts/compress.py (1 hunks)

🧰 Additional context used

🧠 Learnings (6)

📓 Common learnings

Learnt from: quinntaylormitchell
PR: y-scope/clp#1283
File: components/clp-package-utils/clp_package_utils/scripts/compress.py:206-210
Timestamp: 2025-09-04T13:51:02.311Z
Learning: In components/clp-package-utils/clp_package_utils/scripts/compress.py, an empty string ("") is a valid value for the --timestamp-key parameter, so validation should not treat empty strings as missing values.

Learnt from: haiqi96
PR: y-scope/clp#1186
File: components/clp-package-utils/clp_package_utils/scripts/archive_manager.py:40-40
Timestamp: 2025-08-15T15:21:51.607Z
Learning: In components/clp-package-utils/clp_package_utils/scripts/archive_manager.py, the _validate_timestamps function correctly uses Optional[int] for end_ts parameter without a default value because it forces callers to explicitly pass the parsed argument value (which can be None), making the API contract clear and avoiding ambiguity about whether None came from defaults or actual parsed values.

📚 Learning: 2025-09-04T13:51:02.311Z

Learnt from: quinntaylormitchell
PR: y-scope/clp#1283
File: components/clp-package-utils/clp_package_utils/scripts/compress.py:206-210
Timestamp: 2025-09-04T13:51:02.311Z
Learning: In components/clp-package-utils/clp_package_utils/scripts/compress.py, an empty string ("") is a valid value for the --timestamp-key parameter, so validation should not treat empty strings as missing values.

Applied to files:

components/clp-package-utils/clp_package_utils/scripts/compress.py

📚 Learning: 2025-08-15T15:21:51.607Z

Learnt from: haiqi96
PR: y-scope/clp#1186
File: components/clp-package-utils/clp_package_utils/scripts/archive_manager.py:40-40
Timestamp: 2025-08-15T15:21:51.607Z
Learning: In components/clp-package-utils/clp_package_utils/scripts/archive_manager.py, the _validate_timestamps function correctly uses Optional[int] for end_ts parameter without a default value because it forces callers to explicitly pass the parsed argument value (which can be None), making the API contract clear and avoiding ambiguity about whether None came from defaults or actual parsed values.

Applied to files:

components/clp-package-utils/clp_package_utils/scripts/compress.py

📚 Learning: 2025-08-15T15:19:14.562Z

Learnt from: haiqi96
PR: y-scope/clp#1186
File: components/clp-package-utils/clp_package_utils/scripts/archive_manager.py:40-40
Timestamp: 2025-08-15T15:19:14.562Z
Learning: In components/clp-package-utils/clp_package_utils/scripts/archive_manager.py, the _validate_timestamps function correctly uses Optional[int] for end_ts parameter without a default value because it receives parsed argument values that can be None when --end-ts is not provided for the find command.

Applied to files:

components/clp-package-utils/clp_package_utils/scripts/compress.py

📚 Learning: 2025-01-16T16:58:43.190Z

Learnt from: haiqi96
PR: y-scope/clp#651
File: components/clp-package-utils/clp_package_utils/scripts/compress.py:0-0
Timestamp: 2025-01-16T16:58:43.190Z
Learning: In the clp-package compression flow, path validation and error handling is performed at the scheduler level rather than in the compress.py script to maintain simplicity and avoid code duplication.

Applied to files:

components/clp-package-utils/clp_package_utils/scripts/compress.py

📚 Learning: 2025-08-08T06:59:42.436Z

Learnt from: junhaoliao
PR: y-scope/clp#1152
File: components/clp-package-utils/clp_package_utils/scripts/start_clp.py:613-613
Timestamp: 2025-08-08T06:59:42.436Z
Learning: In components/clp-package-utils/clp_package_utils/scripts/start_clp.py, generic_start_scheduler sets CLP_LOGGING_LEVEL using clp_config.query_scheduler.logging_level for both schedulers; compression scheduler should use its own logging level. Tracking via an issue created from PR #1152 discussion.

Applied to files:

components/clp-package-utils/clp_package_utils/scripts/compress.py

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)

GitHub Check: package-image
GitHub Check: lint-check (macos-15)
GitHub Check: lint-check (ubuntu-24.04)

🔇 Additional comments (1)

components/clp-package-utils/clp_package_utils/scripts/compress.py (1)

206-210: LGTM: warning logic is correct and scoped.

Check against None preserves empty-string as valid; placement after CLP_S dataset validation is appropriate; behaviour unchanged.

Pre-merge checks (2 passed, 1 warning)

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run `@coderabbitai generate docstrings` to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The title clearly follows the Conventional Commits format and succinctly captures the primary change—adding a warning when the `--timestamp-key` flag is omitted during compression with the `clp-s` storage engine—without extraneous detail or noise.
Description Check	✅ Passed	The pull request description clearly explains the purpose of the change by detailing the addition of a warning when the `--timestamp-key` flag is omitted, outlines the exact warning text, and summarises the unchanged compression behaviour and validation steps, all of which directly relate to the changeset.

✨ Finishing Touches

📝 Generate Docstrings

🧪 Generate unit tests

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

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

coderabbitai

Actionable comments posted: 1

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

components/clp-package-utils/clp_package_utils/scripts/compress.py (1)

91-96: Don’t forward empty/whitespace --timestamp-key to the native compress command.

Guard against '' or ' ' and trim before passing.

-    if parsed_args.timestamp_key is not None:
-        compress_cmd.append("--timestamp-key")
-        compress_cmd.append(parsed_args.timestamp_key)
+    ts_key = parsed_args.timestamp_key.strip() if parsed_args.timestamp_key is not None else None
+    if ts_key:
+        compress_cmd.append("--timestamp-key")
+        compress_cmd.append(ts_key)

📜 Review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

💡 Knowledge Base configuration:

MCP integration is disabled by default for public repositories
Jira integration is disabled by default for public repositories
Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between f91f98c and 601def0.

📒 Files selected for processing (1)

components/clp-package-utils/clp_package_utils/scripts/compress.py (1 hunks)

🧰 Additional context used

🧠 Learnings (3)

📓 Common learnings

Learnt from: haiqi96
PR: y-scope/clp#1186
File: components/clp-package-utils/clp_package_utils/scripts/archive_manager.py:40-40
Timestamp: 2025-08-15T15:21:51.607Z
Learning: In components/clp-package-utils/clp_package_utils/scripts/archive_manager.py, the _validate_timestamps function correctly uses Optional[int] for end_ts parameter without a default value because it forces callers to explicitly pass the parsed argument value (which can be None), making the API contract clear and avoiding ambiguity about whether None came from defaults or actual parsed values.

📚 Learning: 2025-08-15T15:21:51.607Z

Learnt from: haiqi96
PR: y-scope/clp#1186
File: components/clp-package-utils/clp_package_utils/scripts/archive_manager.py:40-40
Timestamp: 2025-08-15T15:21:51.607Z
Learning: In components/clp-package-utils/clp_package_utils/scripts/archive_manager.py, the _validate_timestamps function correctly uses Optional[int] for end_ts parameter without a default value because it forces callers to explicitly pass the parsed argument value (which can be None), making the API contract clear and avoiding ambiguity about whether None came from defaults or actual parsed values.

Applied to files:

components/clp-package-utils/clp_package_utils/scripts/compress.py

📚 Learning: 2025-08-15T15:19:14.562Z

Learnt from: haiqi96
PR: y-scope/clp#1186
File: components/clp-package-utils/clp_package_utils/scripts/archive_manager.py:40-40
Timestamp: 2025-08-15T15:19:14.562Z
Learning: In components/clp-package-utils/clp_package_utils/scripts/archive_manager.py, the _validate_timestamps function correctly uses Optional[int] for end_ts parameter without a default value because it receives parsed argument values that can be None when --end-ts is not provided for the find command.

Applied to files:

components/clp-package-utils/clp_package_utils/scripts/compress.py

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)

GitHub Check: package-image
GitHub Check: lint-check (ubuntu-24.04)
GitHub Check: lint-check (macos-15)

coderabbitai

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

💡 Knowledge Base configuration:

MCP integration is disabled by default for public repositories
Jira integration is disabled by default for public repositories
Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 601def0 and 444ec3f.

📒 Files selected for processing (1)

components/clp-package-utils/clp_package_utils/scripts/compress.py (1 hunks)

🧰 Additional context used

🧠 Learnings (5)

📓 Common learnings

Learnt from: quinntaylormitchell
PR: y-scope/clp#1283
File: components/clp-package-utils/clp_package_utils/scripts/compress.py:206-210
Timestamp: 2025-09-04T13:51:02.278Z
Learning: In components/clp-package-utils/clp_package_utils/scripts/compress.py, an empty string ("") is a valid value for the --timestamp-key parameter, so validation should not treat empty strings as missing values.

Learnt from: haiqi96
PR: y-scope/clp#1186
File: components/clp-package-utils/clp_package_utils/scripts/archive_manager.py:40-40
Timestamp: 2025-08-15T15:21:51.607Z
Learning: In components/clp-package-utils/clp_package_utils/scripts/archive_manager.py, the _validate_timestamps function correctly uses Optional[int] for end_ts parameter without a default value because it forces callers to explicitly pass the parsed argument value (which can be None), making the API contract clear and avoiding ambiguity about whether None came from defaults or actual parsed values.

📚 Learning: 2025-09-04T13:51:02.278Z

Learnt from: quinntaylormitchell
PR: y-scope/clp#1283
File: components/clp-package-utils/clp_package_utils/scripts/compress.py:206-210
Timestamp: 2025-09-04T13:51:02.278Z
Learning: In components/clp-package-utils/clp_package_utils/scripts/compress.py, an empty string ("") is a valid value for the --timestamp-key parameter, so validation should not treat empty strings as missing values.

Applied to files:

components/clp-package-utils/clp_package_utils/scripts/compress.py

📚 Learning: 2025-08-15T15:21:51.607Z

Learnt from: haiqi96
PR: y-scope/clp#1186
File: components/clp-package-utils/clp_package_utils/scripts/archive_manager.py:40-40
Timestamp: 2025-08-15T15:21:51.607Z
Learning: In components/clp-package-utils/clp_package_utils/scripts/archive_manager.py, the _validate_timestamps function correctly uses Optional[int] for end_ts parameter without a default value because it forces callers to explicitly pass the parsed argument value (which can be None), making the API contract clear and avoiding ambiguity about whether None came from defaults or actual parsed values.

Applied to files:

components/clp-package-utils/clp_package_utils/scripts/compress.py

📚 Learning: 2025-08-15T15:19:14.562Z

Learnt from: haiqi96
PR: y-scope/clp#1186
File: components/clp-package-utils/clp_package_utils/scripts/archive_manager.py:40-40
Timestamp: 2025-08-15T15:19:14.562Z
Learning: In components/clp-package-utils/clp_package_utils/scripts/archive_manager.py, the _validate_timestamps function correctly uses Optional[int] for end_ts parameter without a default value because it receives parsed argument values that can be None when --end-ts is not provided for the find command.

Applied to files:

components/clp-package-utils/clp_package_utils/scripts/compress.py

📚 Learning: 2025-01-16T16:58:43.190Z

Learnt from: haiqi96
PR: y-scope/clp#651
File: components/clp-package-utils/clp_package_utils/scripts/compress.py:0-0
Timestamp: 2025-01-16T16:58:43.190Z
Learning: In the clp-package compression flow, path validation and error handling is performed at the scheduler level rather than in the compress.py script to maintain simplicity and avoid code duplication.

Applied to files:

components/clp-package-utils/clp_package_utils/scripts/compress.py

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)

GitHub Check: package-image
GitHub Check: lint-check (macos-15)
GitHub Check: lint-check (ubuntu-24.04)

🔇 Additional comments (1)

components/clp-package-utils/clp_package_utils/scripts/compress.py (1)

206-210: LGTM: Warning is scoped correctly and respects empty-string semantics.

The None-only check matches the retrieved learning that "" is a valid value for --timestamp-key. Placement under the CLP_S branch is appropriate.

coderabbitai

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

💡 Knowledge Base configuration:

MCP integration is disabled by default for public repositories
Jira integration is disabled by default for public repositories
Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 444ec3f and ff1e162.

📒 Files selected for processing (1)

components/clp-package-utils/clp_package_utils/scripts/compress.py (1 hunks)

🧰 Additional context used

🧠 Learnings (6)

📓 Common learnings

Learnt from: quinntaylormitchell
PR: y-scope/clp#1283
File: components/clp-package-utils/clp_package_utils/scripts/compress.py:206-210
Timestamp: 2025-09-04T13:51:02.278Z
Learning: In components/clp-package-utils/clp_package_utils/scripts/compress.py, an empty string ("") is a valid value for the --timestamp-key parameter, so validation should not treat empty strings as missing values.

Learnt from: haiqi96
PR: y-scope/clp#1186
File: components/clp-package-utils/clp_package_utils/scripts/archive_manager.py:40-40
Timestamp: 2025-08-15T15:21:51.607Z
Learning: In components/clp-package-utils/clp_package_utils/scripts/archive_manager.py, the _validate_timestamps function correctly uses Optional[int] for end_ts parameter without a default value because it forces callers to explicitly pass the parsed argument value (which can be None), making the API contract clear and avoiding ambiguity about whether None came from defaults or actual parsed values.

📚 Learning: 2025-09-04T13:51:02.278Z

Learnt from: quinntaylormitchell
PR: y-scope/clp#1283
File: components/clp-package-utils/clp_package_utils/scripts/compress.py:206-210
Timestamp: 2025-09-04T13:51:02.278Z
Learning: In components/clp-package-utils/clp_package_utils/scripts/compress.py, an empty string ("") is a valid value for the --timestamp-key parameter, so validation should not treat empty strings as missing values.

Applied to files:

components/clp-package-utils/clp_package_utils/scripts/compress.py

📚 Learning: 2025-08-15T15:21:51.607Z

Learnt from: haiqi96
PR: y-scope/clp#1186
File: components/clp-package-utils/clp_package_utils/scripts/archive_manager.py:40-40
Timestamp: 2025-08-15T15:21:51.607Z
Learning: In components/clp-package-utils/clp_package_utils/scripts/archive_manager.py, the _validate_timestamps function correctly uses Optional[int] for end_ts parameter without a default value because it forces callers to explicitly pass the parsed argument value (which can be None), making the API contract clear and avoiding ambiguity about whether None came from defaults or actual parsed values.

Applied to files:

components/clp-package-utils/clp_package_utils/scripts/compress.py

📚 Learning: 2025-08-15T15:19:14.562Z

Learnt from: haiqi96
PR: y-scope/clp#1186
File: components/clp-package-utils/clp_package_utils/scripts/archive_manager.py:40-40
Timestamp: 2025-08-15T15:19:14.562Z
Learning: In components/clp-package-utils/clp_package_utils/scripts/archive_manager.py, the _validate_timestamps function correctly uses Optional[int] for end_ts parameter without a default value because it receives parsed argument values that can be None when --end-ts is not provided for the find command.

Applied to files:

components/clp-package-utils/clp_package_utils/scripts/compress.py

📚 Learning: 2025-01-16T16:58:43.190Z

Learnt from: haiqi96
PR: y-scope/clp#651
File: components/clp-package-utils/clp_package_utils/scripts/compress.py:0-0
Timestamp: 2025-01-16T16:58:43.190Z
Learning: In the clp-package compression flow, path validation and error handling is performed at the scheduler level rather than in the compress.py script to maintain simplicity and avoid code duplication.

Applied to files:

components/clp-package-utils/clp_package_utils/scripts/compress.py

📚 Learning: 2025-08-08T06:59:42.436Z

Learnt from: junhaoliao
PR: y-scope/clp#1152
File: components/clp-package-utils/clp_package_utils/scripts/start_clp.py:613-613
Timestamp: 2025-08-08T06:59:42.436Z
Learning: In components/clp-package-utils/clp_package_utils/scripts/start_clp.py, generic_start_scheduler sets CLP_LOGGING_LEVEL using clp_config.query_scheduler.logging_level for both schedulers; compression scheduler should use its own logging level. Tracking via an issue created from PR #1152 discussion.

Applied to files:

components/clp-package-utils/clp_package_utils/scripts/compress.py

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)

GitHub Check: package-image
GitHub Check: lint-check (macos-15)
GitHub Check: lint-check (ubuntu-24.04)

🔇 Additional comments (1)

components/clp-package-utils/clp_package_utils/scripts/compress.py (1)

206-209: LGTM: Correct scope and copy; respects empty-string semantics.

The warning is gated to CLP_S, checks only for None (so "" remains valid per learnings), and aligns wording with CLI help (“field”). Nice removal of the f-string.

…key` flag when compressing with the `clp-s` storage engine (y-scope#1283) Co-authored-by: Junhao Liao <junhao.liao@yscope.com>

quinntaylormitchell added 2 commits September 4, 2025 08:50

Add warning message if compress.sh script is run without timestamp-ke…

9586628

…y flag

Lint

601def0

quinntaylormitchell requested a review from a team as a code owner September 4, 2025 13:41

coderabbitai Bot reviewed Sep 4, 2025

View reviewed changes

Comment thread components/clp-package-utils/clp_package_utils/scripts/compress.py Outdated

Rabbit

444ec3f

junhaoliao self-requested a review September 4, 2025 13:53

coderabbitai Bot reviewed Sep 4, 2025

View reviewed changes

Comment thread components/clp-package-utils/clp_package_utils/scripts/compress.py Outdated

quinntaylormitchell added 2 commits September 4, 2025 13:58

Rabbit

36f4c2f

Rabbit

ff1e162

coderabbitai Bot reviewed Sep 4, 2025

View reviewed changes

Comment thread components/clp-package-utils/clp_package_utils/scripts/compress.py

junhaoliao reviewed Sep 4, 2025

View reviewed changes

Comment thread components/clp-package-utils/clp_package_utils/scripts/compress.py Outdated

junhaoliao and others added 2 commits September 8, 2025 18:06

Merge branch 'main' into warn-user-timestamp

1c543b8

Emphasize consequences of no timestamp-key flag in message

e6cb592

quinntaylormitchell requested a review from junhaoliao September 9, 2025 16:35

junhaoliao approved these changes Sep 9, 2025

View reviewed changes

quinntaylormitchell merged commit d491e59 into y-scope:main Sep 9, 2025
21 checks passed

junhaoliao mentioned this pull request Nov 4, 2025

Missing results from archives without authoritative timestamp key #1201

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(clp-package): Warn the user if they do not use the `--timestamp-key` flag when compressing with the `clp-s` storage engine#1283

feat(clp-package): Warn the user if they do not use the `--timestamp-key` flag when compressing with the `clp-s` storage engine#1283
quinntaylormitchell merged 7 commits into
y-scope:mainfrom
quinntaylormitchell:warn-user-timestamp

quinntaylormitchell commented Sep 4, 2025 •

edited

Loading

Uh oh!

coderabbitai Bot commented Sep 4, 2025 •

edited

Loading

Uh oh!

coderabbitai Bot left a comment

Uh oh!

Uh oh!

coderabbitai Bot left a comment

Uh oh!

Uh oh!

coderabbitai Bot left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

quinntaylormitchell commented Sep 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Checklist

Validation performed

Summary by CodeRabbit

Uh oh!

coderabbitai Bot commented Sep 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Pre-merge checks (2 passed, 1 warning)

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

quinntaylormitchell commented Sep 4, 2025 •

edited

Loading

coderabbitai Bot commented Sep 4, 2025 •

edited

Loading