@techdivision/opencode-time-tracking 0.6.0 → 0.7.0

package/CHANGELOG.md

@@ -2,6 +2,26 @@
 
 All notable changes to this project will be documented in this file.
 
+## [0.7.0] - 2025-02-04
+
+### Added
+
+- Add `valid_projects` configuration for JIRA project whitelist
+- Restrict ticket detection to specific projects when configured
+
+### Changed
+
+- Default ticket pattern now requires at least 2 uppercase letters
+- Matches: `PROJ-123`, `SOSO-1`, `AB-99`
+- Does not match: `V-1`, `X-9` (single letter), `UTF-8` (false positive)
+
+## [0.6.1] - 2025-02-02
+
+### Fixed
+
+- Fix `ignored_agents` matching to accept agent names with or without `@` prefix
+- Both `"time-tracking"` and `"@time-tracking"` now work in configuration
+
 ## [0.6.0] - 2025-01-31
 
 ### Breaking Changes

package/README.md

@@ -100,6 +100,20 @@ Skip time tracking for specific agents:
 }
 ```
 
+#### Project Whitelist
+
+Restrict ticket detection to specific JIRA projects:
+
+```json
+{
+  "time_tracking": {
+    "csv_file": "...",
+    "global_default": { ... },
+    "valid_projects": ["PROJ", "SOSO", "FEAT"]
+  }
+}
+```
+
 ### Full Example
 
 ```json
@@ -120,11 +134,40 @@ Skip time tracking for specific agents:
         "issue_key": "PROJ-REVIEW"
       }
     },
-    "ignored_agents": ["@internal"]
+    "ignored_agents": ["@internal"],
+    "valid_projects": ["PROJ", "SOSO"]
   }
 }
 ```
 
+## Ticket Detection
+
+### Pattern
+
+By default, tickets must have at least 2 uppercase letters followed by a number:
+- Matches: `PROJ-123`, `SOSO-1`, `AB-99`
+- Does not match: `V-1`, `X-9` (single letter), `UTF-8` (common false positive)
+
+### Project Whitelist
+
+When `valid_projects` is configured, only tickets from those projects are recognized:
+
+```json
+{
+  "time_tracking": {
+    "valid_projects": ["PROJ", "SOSO", "FEAT"]
+  }
+}
+```
+
+With whitelist:
+- Matches: `PROJ-123`, `SOSO-1`, `FEAT-99`
+- Does not match: `UTF-8`, `ISO-9001`, `OTHER-123`
+
+Without whitelist (default):
+- Matches any pattern with 2+ uppercase letters: `PROJ-123`, `AB-1`
+- Does not match single-letter prefixes: `V-1`, `X-99`
+
 ## Fallback Hierarchy
 
 ### Ticket Resolution

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@techdivision/opencode-time-tracking",
-  "version": "0.6.0",
+  "version": "0.7.0",
   "description": "Automatic time tracking plugin for OpenCode - tracks session duration and tool usage to CSV",
   "main": "src/Plugin.ts",
   "types": "src/Plugin.ts",

package/src/Plugin.ts

@@ -61,7 +61,7 @@ export const plugin: Plugin = async ({
 
   const sessionManager = new SessionManager()
   const csvWriter = new CsvWriter(config, directory)
-  const ticketExtractor = new TicketExtractor(client)
+  const ticketExtractor = new TicketExtractor(client, config.valid_projects)
   const ticketResolver = new TicketResolver(config, ticketExtractor)
 
   const hooks: Hooks = {

package/src/hooks/EventHook.ts

@@ -192,8 +192,13 @@ export function createEventHook(
       // Get agent name if available
       const agentString = session.agent?.name ?? null
 
-      // Check if agent should be ignored
-      if (agentString && config.ignored_agents?.includes(agentString)) {
+      // Check if agent should be ignored (tolerant matching: with or without @ prefix)
+      const normalizedAgent = agentString?.replace(/^@/, "")
+      const isIgnoredAgent = config.ignored_agents?.some(
+        (ignored) => ignored.replace(/^@/, "") === normalizedAgent
+      )
+
+      if (agentString && isIgnoredAgent) {
         await client.tui.showToast({
           body: {
             message: `Time tracking skipped for ${agentString} (ignored agent)`,

package/src/services/TicketExtractor.ts

@@ -7,10 +7,12 @@ import type { OpencodeClient } from "../types/OpencodeClient"
 import type { Todo } from "../types/Todo"
 
 /**
- * Regular expression pattern for Jira ticket references.
- * Matches patterns like "PROJ-123", "ABC-1", "FEATURE-9999".
+ * Default regular expression pattern for Jira ticket references.
+ * Requires at least 2 uppercase letters followed by a dash and digits.
+ * Matches patterns like "PROJ-123", "AB-1", "FEATURE-9999".
+ * Does not match single-letter prefixes like "V-1" or "X-99".
  */
-const TICKET_PATTERN = /([A-Z]+-\d+)/
+const DEFAULT_TICKET_PATTERN = /\b([A-Z]{2,}-\d+)\b/
 
 /**
  * Extracts Jira ticket references from user messages and todos.
@@ -22,18 +24,43 @@ const TICKET_PATTERN = /([A-Z]+-\d+)/
  *
  * Returns the first match found, allowing tickets to be updated
  * when mentioned in later messages.
+ *
+ * If `validProjects` is provided, only tickets from those projects
+ * are recognized. Otherwise, any ticket matching the default pattern
+ * (2+ uppercase letters) is accepted.
  */
 export class TicketExtractor {
   /** OpenCode SDK client */
   private client: OpencodeClient
 
+  /** Compiled regex pattern for ticket matching */
+  private ticketPattern: RegExp
+
   /**
    * Creates a new ticket extractor instance.
    *
    * @param client - The OpenCode SDK client
+   * @param validProjects - Optional whitelist of valid JIRA project keys
+   *
+   * @example
+   * ```typescript
+   * // Accept any ticket with 2+ letter prefix
+   * const extractor = new TicketExtractor(client)
+   *
+   * // Only accept PROJ and SOSO tickets
+   * const extractor = new TicketExtractor(client, ["PROJ", "SOSO"])
+   * ```
    */
-  constructor(client: OpencodeClient) {
+  constructor(client: OpencodeClient, validProjects?: string[]) {
     this.client = client
+
+    if (validProjects && validProjects.length > 0) {
+      // Build pattern that only matches specified projects
+      const projectsRegex = validProjects.join("|")
+      this.ticketPattern = new RegExp(`\\b((?:${projectsRegex})-\\d+)\\b`)
+    } else {
+      this.ticketPattern = DEFAULT_TICKET_PATTERN
+    }
   }
 
   /**
@@ -149,7 +176,7 @@ export class TicketExtractor {
    * @returns The first ticket match, or `null` if not found
    */
   private extractFromText(text: string): string | null {
-    const match = text.match(TICKET_PATTERN)
+    const match = text.match(this.ticketPattern)
 
     return match?.[1] ?? null
   }

package/src/types/TimeTrackingConfig.ts

@@ -51,6 +51,16 @@ export interface TimeTrackingJsonConfig {
    * Agent names should include the "@" prefix (e.g., "@internal").
    */
   ignored_agents?: string[]
+
+  /**
+   * Whitelist of valid JIRA project keys.
+   *
+   * @remarks
+   * If set, only tickets from these projects are recognized.
+   * If not set, any ticket matching the default pattern is accepted.
+   * Project keys should be uppercase with at least 2 letters (e.g., "PROJ", "SOSO").
+   */
+  valid_projects?: string[]
 }
 
 /**