Hacktoberfest 2026: the issues maintainers tagged for October, open and beginner-friendly. Browse Hacktoberfest issues

fix(knowledge): list-promotion-candidates prints a stack trace on a malformed claim inventory

Open Beginner friendly
  • Is something wrong?
#6,982 1 comment 0 reactions 0 assignees View on GitHub

Maintainers usually reply within 1 day

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
82/100
Issue type
Bug
Clarity
Clearly specified
Activity status
Active
Tech stack
javascript, nodejs
Domain
cli

Research direction

Start with list-promotion-candidates.js in plugins/knowledge/skills/video-digest/extraction/watch/: the zero-sessions throw is around lines 45-48 and the CLI block around lines 114-121 has no catch. Copy the error handling from the sibling vision-gated-promote.js (lines 93-107), which writes the message to stderr and exits 1. Done means a malformed inventory prints only the one-line message with the format hint and exits nonzero, with a CLI test that checks both.

Written by the indexing model from the issue text.

Description

good first issue needs-triage priority: low

Summary

Run as a CLI, list-promotion-candidates.js calls listPromotionCandidates with no error handling, so its documented failure (no session parsed from research/claim-inventory.md) and any missing-file error escape as an uncaught exception with a Node stack trace. The sibling vision-gated-promote.js catches, prints the message to stderr and exits 1.

Failure scenario

Expected: a malformed claim inventory prints the one-line message with the format hint to stderr and exits with a nonzero code, with no stack trace.

Actual: the function throws at the zero-sessions check, and the CLI block does not catch it, so Node prints the stack trace. The useful part (SESSION_FORMAT_HINT) is buried under it.

Evidence

Suggested fix

Wrap the call in try/catch, write error.message with writeStderr, and process.exit(1), as vision-gated-promote.js does. Add a CLI test with a claim inventory that parses to zero sessions asserting a one-line stderr and a nonzero exit.

Related

  • #6813 (session-coverage gates and a zero-session inventory)

🤖 Generated with Claude Code

Dominant language
Shell
Stars
22
Forks
2
Avg merge
6h 42m
Merged PRs (30d)
902

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from melodic-software/claude-code-plugins

All issues in melodic-software/claude-code-plugins

Similar issues

More Shell/Bash issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.