[spark-compete] fix: close SQLite database gracefully on SIGINT/SIGTERM

[binance1230]

{
  "schema": "spark-compete-hotfix-v1",
  "event": "spark-compete-first-event",
  "submission_mode": "public_repo_pr",
  "submission_target_url": "https://github.com/vibeforge1111/spark-telegram-bot/pull/839",
  "team": {
    "name": "king",
    "members": [
      "@king1005678",
      "@AtwoodJonathan1",
      "@JaredAddison12"
    ],
    "llm_device_holder": "@king1005678",
    "device_holder_github": "binance1230",
    "github_accounts": [
      "binance1230"
    ]
  },
  "target_repo": {
    "id": "vibeforge1111/spark-telegram-bot",
    "source": "https://github.com/vibeforge1111/spark-telegram-bot",
    "owner_surface": "telegram-bot"
  },
  "issue": {
    "type": "bug",
    "severity": "high",
    "title": "SQLite database not closed on SIGINT/SIGTERM \u2014 risks WAL corruption",
    "actual_behavior": "SIGINT/SIGTERM handlers call releaseGatewayOwnership and stop the bot, but do not close the SQLite database. This risks WAL corruption on unclean shutdown.",
    "expected_behavior": "On shutdown signals, the database should be gracefully closed: checkpoint the WAL, then close the connection.",
    "repro_steps": [
      "1. Start the bot and let it open the SQLite database.",
      "2. Send SIGINT or SIGTERM to the process.",
      "3. Observe that the database is not closed \u2014 the WAL file may be left in an inconsistent state."
    ],
    "affected_workflow": "Telegram bot shutdown and data integrity"
  },
  "evidence": {
    "safe_links_only": true,
    "before_after_proof": "Before: SIGINT/SIGTERM handlers do not close SQLite. After: closeJsonState() checkpoints WAL and closes the connection before the process exits.",
    "links": [
      "https://github.com/vibeforge1111/spark-telegram-bot/pull/839"
    ],
    "forbidden": [
      "No raw logs, private conversations, secrets, tokens, screenshots of private chats, or archive downloads."
    ]
  },
  "proposed_fix": {
    "approach": "Add closeJsonState() function that checkpoints WAL (PRAGMA wal_checkpoint(TRUNCATE)) and closes the SQLite connection. Call it from both SIGINT and SIGTERM signal handlers.",
    "files_expected": [
      "src/jsonState.ts",
      "src/index.ts"
    ],
    "tests_or_smoke": "smoke_test: sending SIGINT/SIGTERM triggers closeJsonState() which checkpoints WAL and closes the db"
  },
  "pr": {
    "url": "https://github.com/vibeforge1111/spark-telegram-bot/pull/839",
    "branch": "fix/db-graceful-shutdown",
    "title_prefix": "[spark-compete]",
    "author_github": "binance1230",
    "body_must_include": [
      "packet",
      "team",
      "pr_author",
      "repo",
      "actual_behavior",
      "expected_behavior",
      "repro_steps",
      "before_after_proof",
      "tests_or_smoke",
      "duplicate_notes",
      "risk_notes",
      "review_claim"
    ],
    "replaces_url": "https://github.com/vibeforge1111/spark-telegram-bot/pull/87"
  },
  "review_claim": {
    "impact_claim": "high",
    "evidence_types": [
      "smoke_test",
      "redacted_conversation_excerpt"
    ],
    "duplicate_notes": "Replaces #87 (which contained stacked fixes). This PR addresses only the DB shutdown root cause.",
    "risk_notes": "Low risk: closeJsonState() is best-effort with try/catch/finally. If database is not open, it returns immediately. No behavioral change to normal operation.",
    "review_state_requested": "pr_review"
  }
}