Skip to content

Add test for buildSystemPrompt with non-empty style profile #146

Description

@404-Page-Found

Description

The existing test in \ ests/build-system-prompt.test.mjs\ only tests \�uildSystemPrompt\ with an empty profile (totalCommits = 0). The \�uildStyleGuidance\ function has many conditional branches that are never tested:

  • Imperative mood guidance
  • Sentence case guidance
  • Common prefix guidance with rate >= 30%
  • Multiple prefix listing
  • Scope usage guidance
  • Body usage guidance

Proposed Solution

Add a new test that provides a profile with non-zero values for all these fields and asserts the prompt includes the expected guidance strings.

Why It's a Good First Issue

The file already exists with one test. Adding a second test case with a populated profile exercises the main logic of the style guidance builder. Pure function, no I/O, clear expected outputs.

Files

  • \ ests/build-system-prompt.test.mjs\
  • \src/llm/prompt.ts\ (lines 3-43)

Metadata

Metadata

Assignees

No one assigned

    Type

    No fields configured for Task.

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions