Skip to content

feat(signals): classify issue-discovery lifecycle - #124

Merged
JSONbored merged 4 commits into
JSONbored:mainfrom
oktofeesh1:codex/issue-109-lifecycle
May 30, 2026
Merged

feat(signals): classify issue-discovery lifecycle#124
JSONbored merged 4 commits into
JSONbored:mainfrom
oktofeesh1:codex/issue-109-lifecycle

Conversation

@oktofeesh1

@oktofeesh1 oktofeesh1 commented May 30, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add deterministic issue-discovery lifecycle states
  • include lifecycle state in issue-quality reports
  • distinguish solved, valid solved, stale, duplicate, invalid, and closed-without-solver issues

What changed

  • classifies lifecycle from issue state, labels, linked PRs, and recent merged PR context
  • keeps direct-PR-first repos from encouraging issue filing through lifecycle warnings
  • updates schema and focused tests for lifecycle state propagation
  • keeps readiness integration fixtures current so freshness SLO assertions remain deterministic in CI

Why

  • issue-quality guidance needs to separate valid solved issue-discovery activity from raw closed or stale issue activity.

Validation

  • git diff --check
  • npx tsc --noEmit
  • npx vitest run test/unit/signals-v2.test.ts test/unit/local-branch.test.ts
  • npm run test:coverage
  • GitHub validate, Contributor trust, and Superagent Security Scan pass on the latest head

Closes #109

Add deterministic issue lifecycle states and include them in issue-quality reports so solved, stale, duplicate, and invalid issue-discovery activity is separated.
@oktofeesh1
oktofeesh1 requested a review from JSONbored as a code owner May 30, 2026 05:21
@dosubot dosubot Bot added the size:M label May 30, 2026

@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: 8719964748

ℹ️ About Codex in GitHub

Codex has been enabled to automatically 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 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread src/signals/engine.ts Outdated

@JSONbored JSONbored left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

@oktofeesh1 this is merge-ready from my pass.

  • Deterministic: check rollup is green and merge-tree against current origin/main is clean.
  • Maintainer judgment: lifecycle classification no longer treats broad bounty/reward wording as completion; it relies on issue state, labels, and cached linked solver PR evidence.
  • Validation: tests cover stale, duplicate, invalid, closed-without-solver, solved, valid-solved, and unverified PR mentions.

@dosubot dosubot Bot added the lgtm label May 30, 2026
@JSONbored
JSONbored merged commit da6f71a into JSONbored:main May 30, 2026
3 checks passed
@JSONbored JSONbored added gittensor:feature Gittensor-scored feature linked to a feature issue — scores a 0.25x multiplier. and removed feature labels Jun 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gittensor:feature Gittensor-scored feature linked to a feature issue — scores a 0.25x multiplier.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(signals): model issue-discovery lifecycle state separately

2 participants