simp / simp/puppetsync

setup_project_repos prints the repo-targets table twice

Open Beginner friendly
#81 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug
Dominant language
Ruby
Stars
1
Forks
5
Avg merge
8d 1h
Merged PRs (30d)
4

Description

puppetsync::setup_project_repos emits the repo-targets summary table twice at session start:

out::message(puppetsync::summarize_repo_targets($repos))
warning(puppetsync::summarize_repo_targets($repos,true))

Once plain via out::message and once colorized via warning() — and since Bolt's default console level shows warnings, both render in every run.

Fix: keep one (probably the colorized out::message form, matching how output_pipeline_results renders the results table) and drop the other.

Not to be confused with the whole-output doubling under --log-level info: that one is Bolt behavior (out::message prints directly to the console and emits an info-level log event, so raising the console level to info renders everything twice) — reproduced 1× at default vs 2× at info on a minimal fixture; nothing puppetsync can cleanly do about it, and it only occurs when verbose output was explicitly requested.

Cosmetic; fine to fold into whatever PR next touches setup_project_repos.pp rather than fixing standalone.

🤖 Generated with Claude Code

Contributor guide

No contributing guide indexed for this repository

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.

Research direction

Start in setup_project_repos.pp and compare its repo-targets output with the rendering used by output_pipeline_results. Confirm the session-start summary is emitted only once, then run the relevant project checks or fixture to verify the duplicate table is gone.

Written by the indexing model from the issue text.

Assessment

Domain
tooling
Issue type
Bug
Difficulty
1/5
Estimated time
Under an hour
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
75/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.