Skip to content

Decide and document zero-day heartbeat threshold behavior #42

Description

@theDakshJaitly

Follow-up from Copilot review on PR #39.

Problem

heartbeat.staleDays, memoryCleanupDays, and dailyMemoryRetentionDays currently use positive-number validation, so 0 is rejected and the default is used instead.

That means config cannot express: consider stale as soon as older than today.

Decision needed

Choose one of these behaviors:

  1. Allow 0 for day-based heartbeat thresholds while still rejecting negatives.
  2. Keep requiring positive values, but document that 0 is intentionally unsupported.

Acceptance criteria

  • The intended behavior is documented in README/config docs.
  • Tests reflect the chosen behavior.
  • If 0 is allowed, config parsing accepts >= 0 for day-based heartbeat thresholds only.

Source: #39 (comment)

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions