Merge pull request #177 from DiogoRibeiro7/codex/add-overdue-todo-detection-feature

Add overdue TODO warnings

Author: DiogoRibeiro7

action.yml

@@ -17,6 +17,11 @@ inputs:
     description: Use structured tag extraction with @assignee, #module, and key=value
     default: 'false'
 
+  warn-overdue:
+    required: false
+    description: Emit warnings for TODOs with due dates in the past
+    default: 'false'
+
   issue-title-template:
     required: false
     description: Optional path to custom issue title template

src/ActionMain.ts

@@ -7,7 +7,7 @@ import { extractTodosFromDir } from './parser/extractTodosFromDir';
 import { extractTodosWithStructuredTagsFromDir } from './parser/extractTodosWithStructuredTagsFromDir'; // 👈 novo
 import { TodoItem } from './parser/types';
 import { getExistingIssueTitles, createIssueIfNeeded } from './core/issueManager';
-import { generateMarkdownReport } from './core/report';
+import { generateMarkdownReport, warnOverdueTodos } from './core/report';
 import { limitTodos, todoKey } from './core/todoUtils';
 import { generateChangelogFromTodos } from './core/changelog';
 
@@ -30,6 +30,8 @@ async function run(): Promise<void> {
 
     const useStructured = core.getInput('structured') === 'true';
 
+    const warnOverdue = core.getInput('warn-overdue') === 'true';
+
     const todos: TodoItem[] = useStructured
       ? extractTodosWithStructuredTagsFromDir(workspace)
       : extractTodosFromDir(workspace);
@@ -48,6 +50,10 @@ async function run(): Promise<void> {
       return true;
     });
 
+    if (warnOverdue) {
+      warnOverdueTodos(uniqueTodos);
+    }
+
     const issueLimit = parseInt(core.getInput('limit') || '5', 10);
     const todosToCreate = limitTodos(uniqueTodos, issueLimit);
 

src/core/report.ts

@@ -2,6 +2,7 @@ import fs from 'fs';
 import path from 'path';
 import { TodoItem } from '../parser/types';
 import { todoKey } from './todoUtils';
+import * as core from '@actions/core';
 
 const PRIORITY_ORDER = ['high', 'medium', 'low'];
 
@@ -13,6 +14,21 @@ function getDue(todo: TodoItem): string {
   return todo.metadata?.due ?? '';
 }
 
+export function findOverdueTodos(todos: TodoItem[]): TodoItem[] {
+  const today = new Date().toISOString().split('T')[0];
+  return todos.filter(t => {
+    const due = t.metadata?.due;
+    return typeof due === 'string' && due < today;
+  });
+}
+
+export function warnOverdueTodos(todos: TodoItem[]): void {
+  const overdue = findOverdueTodos(todos);
+  for (const todo of overdue) {
+    core.warning(`\u23F0 Overdue TODO (${todo.metadata!.due}): ${todo.text} (${todo.file}:${todo.line})`);
+  }
+}
+
 function sortTodos(a: TodoItem, b: TodoItem): number {
   const pa = getPriority(a);
   const pb = getPriority(b);

tests/overdueDetection.test.ts

@@ -0,0 +1,25 @@
+import { describe, it, expect } from 'vitest';
+import { findOverdueTodos } from '../src/core/report';
+import { TodoItem } from '../src/parser/types';
+
+function format(date: Date) {
+  return date.toISOString().split('T')[0];
+}
+
+describe('findOverdueTodos', () => {
+  it('detects todos with past due dates', () => {
+    const yesterday = format(new Date(Date.now() - 86400000));
+    const today = format(new Date());
+    const tomorrow = format(new Date(Date.now() + 86400000));
+    const todos: TodoItem[] = [
+      { tag: 'TODO', text: 'past', file: 'a.ts', line: 1, metadata: { due: yesterday } },
+      { tag: 'TODO', text: 'today', file: 'b.ts', line: 2, metadata: { due: today } },
+      { tag: 'TODO', text: 'future', file: 'c.ts', line: 3, metadata: { due: tomorrow } },
+      { tag: 'TODO', text: 'none', file: 'd.ts', line: 4 }
+    ];
+
+    const result = findOverdueTodos(todos);
+    expect(result.length).toBe(1);
+    expect(result[0].text).toBe('past');
+  });
+});