Add unit test: show --json includes structured audit (present & absent)

[SorraTheOrc]

Headline
Ensure wl show --json includes an audit object only when audit data exists, and add unit tests for presence and absence cases.

Problem statement
Ensure wl show --json returns a stable, typed JSON object that always contains a workItem object and conditionally includes an audit object only when audit data exists. Add unit tests that verify both the presence and absence cases so automation can rely on the shape without additional heuristics.

Users

• Test authors and maintainers of the Worklog CLI who rely on stable, predictable JSON output from wl show --json for automation and tooling.

Example user stories

• As a test author, I want wl show --json to include an audit key only when audit data exists so downstream consumers can parse the output without extra heuristics.
• As an automation consumer, I want a typed/structured JSON output so I can reliably read workItem and optional audit fields in scripts.

Success criteria

• Unit tests added that assert wl show --json returns a JSON object with workItem and when audit data is present includes an audit object with time, author, and text fields.
• Unit tests added that assert when audit is absent the audit property is not present in the JSON output (not null or empty object).
• The new tests are run in CI and pass locally; full project test suite passes with the new changes.
• Related documentation (README or CLI docs) updated to document the --json output shape.

Constraints

• Do not change existing public JSON keys other than adding/omitting audit as specified (backwards compatibility required for other fields).
• Tests should be hermetic and not rely on external services; use in-repo fixtures or mocks.

Existing state

• Current work item: WL-0MN7ZP9GX001XJE4 (title: "Add unit test: show --json includes structured audit (present & absent)") exists and is assigned to Map at stage idea / status in-progress.
• Parent: WL-0MMRJM03Q0BG25IG appears related per the work item metadata.

Desired change

• Implement unit tests that cover both presence and absence of audit in the wl show --json output and ensure tests are added to the appropriate test files and run as part of the test suite.
• Update CLI documentation to describe the --json output shape and the conditional audit field.

Related work

• Parent work item: WL-0MMRJM03Q0BG25IG (related epic/feature).
• Evidence from repository/worklog: work item WL-0MN7ZP9GX001XJE4 comments indicate a PR was opened (PR WL-0MN7ZP9GX001XJE4: add show --json audit presence/absence test #1167) and commits e1996ff and 5544bc9 reference tests and helpers updates. A test file tests/cli/show-json-audit.test.ts is referenced in the work item comments. (Source: wl show WL-0MN7ZP9GX001XJE4 --json comments.)

Appendix: Clarifying questions & answers

• Q: "Should I ask any clarifying questions about the work?" — Answer (user instruction): "do not ask questions". Source: seed input. Final: yes.
• Q: "Are there mandatory formatting or additional JSON keys required by downstream consumers?" — Answer (agent inference): Not provided by the user or repo artifacts; assume only the audit key behaviour described in the seed context is required. Source: agent inference from WL-0MN7ZP9GX001XJE4 work item description and comments. Final: yes.

Risks & assumptions

• Risk: The tests or implementation may already exist (PR opened per comments). Mitigation: verify the mentioned PR/commits (e1996ff, 5544bc9) and, if merged, mark this intake as already implemented or adjust to coverage/regression tests as needed.
• Risk: Changing wl show output shape could break downstream automation. Mitigation: tests must assert backward compatibility for existing fields and only validate the presence/absence behaviour of audit.
• Assumption: Downstream consumers expect the audit key to be omitted entirely when absent (not present as null). If this is incorrect, request clarification. (Current answer: user did not provide - agent inference used.)

[SorraTheOrc]

OpenCode

Added test tests/cli/show-json-audit.test.ts validating 'wl show --json' includes structured audit when present and omits audit key when absent. Commit 5544bc9.

[SorraTheOrc]

OpenCode

Also updated src/commands/helpers.ts to ensure concise/normal human audit excerpts use redactAuditText consistently (commit e1996ff).

[SorraTheOrc]

worklog

Closed with reason: PR #1167 opened with tests and helpers update (see commit e1996ff, 5544bc9)

[SorraTheOrc]

ampa

Automated intake dispatched by AMPA. pid=7508.

[SorraTheOrc]

effort_and_risk_skill

Effort and Risk Report

Effort | Small | 2.17h
Risk | Low | 4/20
Confidence | 71% | unknowns: Whether tests/commit already merged; if merged this becomes validation work

{
  "effort": {
    "unit": "hours",
    "tshirt": "Small",
    "o": 1.0,
    "m": 2.0,
    "p": 4.0,
    "expected": 2.17,
    "recommended": 6.17,
    "range": [
      5.0,
      8.0
    ]
  },
  "risk": {
    "probability": 2.12,
    "impact": 2.12,
    "score": 4,
    "level": "Low",
    "top_drivers": [],
    "mitigations": [
      "Add targeted tests and integration checks",
      "Lock dependencies and add compatibility tests",
      "Schedule extra review for risky components"
    ]
  },
  "confidence_percent": 71,
  "assumptions": [
    "Small, focused test addition; existing helpers may be reused"
  ],
  "unknowns": [
    "Whether tests/commit already merged; if merged this becomes validation work"
  ]
}

[SorraTheOrc]

pi-agent

Plan: created 3 feature work items:

• CLI: show --json audit tests (WL-0MO67S58L0020G38)
• Docs: document show --json output shape (WL-0MO67SAI900878NG)
• CI: verify and gate audit-related tests (WL-0MO67SHPB0005FKG)

Dependencies added:

• CI: verify and gate audit-related tests depends on CLI: show --json audit tests

Plan: changelog:

• 2026-04-19T20:24:13Z created WL-0MO67S58L0020G38
• 2026-04-19T20:24:20Z created WL-0MO67SAI900878NG
• 2026-04-19T20:24:29Z created WL-0MO67SHPB0005FKG
• 2026-04-19T20:24:48Z added dependency WL-0MO67SHPB0005FKG -> WL-0MO67S58L0020G38

Seed context:

• Title: Add unit test: show --json includes structured audit (present & absent) (WL-0MN7ZP9GX001XJE4)
• Type: test; Stage: intake_complete
• One-line: Add unit tests verifying 'wl show --json' includes an 'audit' object only when audit data exists and omits it when absent.

Appendix:

• No new clarifying questions were asked during this planning run. Existing Appendix entries attached to the parent work item were used as authoritative (no duplication).