Skip to content

fix(triage): support level-2 issue headings - #916

Merged
XiaoSeS merged 1 commit into
mainfrom
fix/triage-heading-levels
Oct 8, 2026
Merged

XiaoSeS merged 1 commit into
mainfrom
fix/triage-heading-levels

Conversation

@dongmucat

@dongmucat dongmucat commented Oct 1, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Allow issue triage to parse both level-2 (##) and level-3 (###) section headings.
  • Prevent complete issues written with level-2 headings from being routed to triage/needs-info.
  • Add regression coverage for level-2, level-3 Issue Form, and mixed heading bodies.

Validation

  • Targeted Deno tests passed (3 passed, 0 failed)
  • Deno format check passed
  • Deno lint passed
  • Deno type check passed
  • git diff --check passed
  • API contracts unchanged; no OpenAPI update required
  • No frontend, backend Java, database, or deployment changes

Commands run:

npx --yes deno@2 test --config .github/scripts/deno.json .github/scripts/issue-triage-lib_test.ts
npx --yes deno@2 fmt --config .github/scripts/deno.json --check .github/scripts/issue-triage-lib.ts .github/scripts/issue-triage-lib_test.ts
npx --yes deno@2 lint --config .github/scripts/deno.json .github/scripts/issue-triage-lib.ts .github/scripts/issue-triage-lib_test.ts
npx --yes deno@2 check --config .github/scripts/deno.json .github/scripts/issue-triage-lib.ts .github/scripts/issue-triage-lib_test.ts

Risk

  • User-facing impact: correctly classifies complete issues using ## headings.
  • Deployment or migration impact: none.
  • Rollback approach: revert commit 906a5e6c.

Notes

Signed-off-by: dongmucat <1127093059@qq.com>
@dongmucat
dongmucat force-pushed the fix/triage-heading-levels branch from dbeb57e to 906a5e6 Compare October 1, 2026 18:23

@FenjuFu FenjuFu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

感谢补上回归测试。改动与 #915 一致,额外覆盖了 Issue Form 的 ### 和 ##/### 混用两种情况,比 #915 更完整。LGTM,合入后我会关闭 #915。

@XiaoSeS
XiaoSeS merged commit 8a0d82f into main Oct 8, 2026
7 checks 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.

[Bug] Issue triage ignores level-2 headings and marks filled issues as needs-info

3 participants