Skip to content

Make next action dependency-aware - #43

Merged
Alajmah merged 3 commits into
mainfrom
n42
Jun 6, 2026
Merged

Alajmah merged 3 commits into
mainfrom
n42

Conversation

@Alajmah

@Alajmah Alajmah commented Jun 6, 2026

Copy link
Copy Markdown
Contributor

Summary

Improves next-action recommendation so it selects only tasks whose declared dependencies are complete.

Changes

  • Skips active tasks with missing or active dependencies.
  • Reports when no unblocked active task is available.
  • Preserves overdue and nearest-due prioritization for unblocked tasks.
  • Adds tests for unblocked selection and all-blocked behavior.

Closes #42.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b26df2680a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread tests/test_next_action.py Outdated
report = recommend_next_action(
[
_task("blocked", "Blocked Task", "planned", dependencies=["dependency"], due="2026-06-01"),
_task("dependency", "Dependency", "todo", due="2026-06-02"),

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Fix fixtures so blocked dependency tests pass

In this fixture, dependency is still an active task with no declared dependencies and an earlier due date than ready, so the recommender will include it as an unblocked candidate and select Dependency, not Ready Task. The same setup also means the later “all active tasks are blocked” case is not actually all blocked. With dependencies installed, these new tests fail rather than validating the dependency-aware behavior.

Useful? React with 👍 / 👎.

@Alajmah
Alajmah merged commit 64b1497 into main Jun 6, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve next-action recommendation with dependency readiness

1 participant