Set resync and reindex period as environment variable

[ErwanLeGallKosmos]

Hi
I needed to be able to modify the ReSync and the ReIndex period value with environment variables.
You may also be interested in this.

Summary by CodeRabbit

• New Features

  • Introduced dynamic configuration for background task intervals using environment settings, allowing for customizable timing with default values.

• Chore

  • Streamlined internal configuration management to improve maintainability and flexibility without altering user-facing functionality.
  • Updated import statements to include new environment constants for better configuration handling.

[coderabbitai]

Actionable comments posted: 2

🧹 Nitpick comments (1)

packages/backend/src/environment.ts (1)

31-32: Document the environment variables.

Add JSDoc comments to document the purpose and expected format of these environment variables.

Add documentation above the constants:

/**
 * Interval in milliseconds between repository reindexing operations.
 * Defaults to 1 hour if not specified.
 * @example
 * REINDEX_INTERVAL_MS=3600000
 */
export const REINDEX_INTERVAL_MS = // ...

/**
 * Interval in milliseconds between configuration resynchronization operations.
 * Defaults to 24 hours if not specified.
 * @example
 * RESYNC_CONFIG_INTERVAL_MS=86400000
 */
export const RESYNC_CONFIG_INTERVAL_MS = // ...

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 39e761f and 7af729a.

📒 Files selected for processing (3)

• packages/backend/src/constants.ts (1 hunks)
• packages/backend/src/environment.ts (2 hunks)
• packages/backend/src/main.ts (1 hunks)

✅ Files skipped from review due to trivial changes (1)

• packages/backend/src/main.ts

🔇 Additional comments (1)

packages/backend/src/constants.ts (1)

1-9: LGTM! Clean removal of constants.

The removal of REINDEX_INTERVAL_MS and RESYNC_CONFIG_INTERVAL_MS constants is appropriate as they're being moved to the environment configuration.

[msukkari]

Thanks for this! Out of curiosity was there a reason that you wanted to modify these values using an env var vs. the currently supported way of setting the values in the config? https://sourcebot.dev/search/browse/github.com/sourcebot-dev/sourcebot@HEAD/-/blob/packages/backend/src/main.ts?highlightRange=210%3A1%2C216%3A6