**Commit Check failed**
>
-> ❌ **2 of 4 checks failed**
+> 
Pull request: passed
Commits: 1 of 2 failed
-> ():
+> - Branch: change "Feature/Add-Login" to "feature/Add-Login"
+> ```
+>
+> **Commit messages.** Reword it, then force-push:
+>
+> ```bash
+> git commit --amend
+> git push --force-with-lease
+> ```
+>
+> **Branch name.** Rename the branch and push it. A pull request cannot change its branch, so open a new one from the renamed branch:
+>
+> ```bash
+> git branch -m feature/Add-Login
+> git push -u origin HEAD
+> ```
+>
+> **Next time.** The [pre-commit hook](https://commit-check.com/guides/integrations/#as-a-pre-commit-hook) runs the same rules before you push.
+>
+>
+>
+> Show all 4 checks
+>How to fix
+> +> **With an AI coding agent.** Paste this prompt: +> +> ```text +> Fix these Commit Check findings without changing any code: +> - Commit 5584f46 "bad msg": Use
+> ():
> Branch
-> ✖ Branch (1 failure)
+> ✖ branch Feature/Add-Login
> CC201 branch
> value: Feature/Add-Login
> The branch should follow Conventional Branch.
@@ -150,36 +188,28 @@ click away:
>
>
>
-> _commit-check <version> · [Rules reference](https://commit-check.com/rules/)_
+> Checked by [Commit Check](https://commit-check.com) <version> · [Rules](https://commit-check.com/rules/) · [Check before you push](https://commit-check.com/guides/integrations/#as-a-pre-commit-hook)
The rest follow the same layout:
-- **Passed:** one line, `✅ All 3 checks passed`, with the tree folded away.
-- **Warned:** a rule under `warn` gets its own ⚠ row and counts as passed.
+- **Passed:** the heading reads "Commit Check passed", with the list folded away.
+- **Warned:** a rule under `warn` gets its own diff under "Warnings", a ⚠ in
+ the list, and counts as passed.
- **Skipped:** a run where nothing was validated — typically a bot listed in
- `ignore_authors` — reads `⊘`, never `✔`.
+ `ignore_authors` — reads "Commit Check skipped", never passed.
-The step log prints the same tree, plus one annotation per finding on the
-Files changed tab. With `pr-comments: true` the same report is posted to the
-pull request, and later runs edit that one comment instead of adding more.
+The step log prints the same list, each group folded under its own result, plus
+one annotation per finding on the run page. With `pr-comments: true` the same
+report is posted to the pull request, and later runs edit that one comment
+instead of adding more.
With `pr-body-ai-attribution: true` and `ai_attribution = "forbid"`, a
-description that credits an AI co-author fails the same way, and the report
-names the line to remove:
+description that credits an AI co-author fails the same way, and the diff is
+the line to remove:
-> ❌ **1 of 2 checks failed**
->
-> | Scope | Checked value | Failed checks |
-> |---|---|---|
-> | PR description | `Claude Code` | [CC013 ai-attribution](https://commit-check.com/rules/#cc013) |
->
-> ```text
-> Pull request
-> ✖ PR description (1 failure)
-> CC013 ai-attribution
-> value: Claude Code
-> AI attribution is forbidden in this project — detected: Claude Code
-> Suggest: In the description, remove "Co-Authored-By: Claude All 4 checks
> > ```text > Pull request -> ✔ PR title (feat: add login page) -> Commit message -> ✔ Commit 1/2 (d87faca) (feat: add login page) -> ✖ Commit 2/2 (5584f46) (1 failure) +> ✔ title feat: add login page +> Commits +> ✔ d87faca feat: add login page +> ✖ 5584f46 bad msg > CC001 message > value: bad msg > The commit message should follow Conventional Commits. > Suggest: Use![{STATUS_GLYPHS[status]}]({STATUS_MARK_URL}/{name}.png)
'
+ )
+
+def _group_lines(results: list[ScopeResult]) -> str:
+ """One line per group, its status mark in front: the overview."""
+ return "\n".join( + f"{_status_mark(status, 16, line=True)} {group_name}: {text}" + for group_name, scopes in _grouped(results) + for status, text in [_group_status(scopes)] + ) -def _markdown_details(results: list[ScopeResult]) -> str: - """Render the collapsible details block listing every scope. - Mirrors the step log layout (group name, ✔/✖ scope lines with the checked - value) and adds the failure reason and suggestion under each failing rule, - so one expand answers both "what was checked" and "what failed and why". - The same block is used whether or not anything failed — on a clean run the - failure branches simply never fire. +def _hunk(scope: ScopeResult, status: str) -> list[str]: + """One scope's findings as a diff hunk. + + ``@@ where @@``, then red for what was found and has to go, green for + what to write instead, and a ``#`` comment for advice that has no + literal text to offer. GitHub colours all three in a ``diff`` block, + which is the only colour a comment or a job summary can carry. """ - _failed, total = _check_counts(results) - unit = "check" if total == 1 else "checks" - label = f"Show all {total} {unit}" if total else "Show details" - body = _render_tree(results, include_docs=False) - # The tree quotes commit subjects, errors and suggestions as they are. A - # value with three backticks in it (a `docs:` commit showing a fence) - # would close a fixed ``` fence and spill the rest of the report out as - # prose, so the fence is one longer than any backtick run inside. - fence = "`" * max(3, _longest_backtick_run(body) + 1) - lines = ["
", f"
"])
- return "\n".join(lines)
+ is_commit = bool(scope.sha) and scope.label.startswith("Commit")
+ lines = [f"@@ {scope.sha[:7] if is_commit else scope.label} @@"]
+ removed: list[str] = []
+ added: list[str] = []
+ advice: list[str] = []
+ for check in _findings(scope, status):
+ minus, plus = _line_edits(scope, check)
+ removed.extend(minus)
+ added.extend(plus)
+ if not (minus or plus):
+ advice.append(_advice(check))
+ if scope.raw_text and not scope.checks:
+ advice.append("the output could not be parsed, see the step log")
+ removed, added, advice = (list(dict.fromkeys(x)) for x in (removed, added, advice))
+ value = _first_value(scope)
+ if value and value not in removed:
+ # Advice is about the value itself, so it is what was found; a fix
+ # that only touches other lines leaves it as context.
+ lines.append(f"{'-' if advice else ' '} {value}")
+ lines.extend(f"- {line}" for line in removed)
+ lines.extend(f"+ {line}" for line in added)
+ lines.extend(f"# {line}" for line in advice)
+ return lines
-def _longest_backtick_run(lines: list[str]) -> int:
- """Length of the longest run of consecutive backticks across ``lines``."""
- return max((len(m) for line in lines for m in re.findall(r"`+", line)), default=0)
+def _diff_block(results: list[ScopeResult], status: str = "fail") -> list[str]:
+ """The findings of one kind as a ``diff`` block, with their links under it.
+
+ A scope appears when it has an entry of the requested kind — checked via
+ ``scope.failures`` / ``scope.warnings``, not ``scope.status`` — so a scope
+ that both failed and warned shows up in both blocks.
+
+ A long list is cut at ``MAX_HUNKS`` and the rest folded away, so twenty
+ bad commits do not push the rest of the pull request off the screen. A
+ code block cannot hold links, so the commits shown and the rules that
+ spoke get one small line of them underneath.
+ """
+ scopes = [
+ scope
+ for scope in results
+ if _findings(scope, status)
+ or (status == "fail" and scope.raw_text and not scope.checks)
+ ]
+ hunks = [_hunk(scope, status) for scope in scopes]
+ shown = len(hunks) if len(hunks) <= MAX_HUNKS + HUNK_SLACK else MAX_HUNKS
+ lines = _fenced("diff", [line for hunk in hunks[:shown] for line in hunk])
+ if len(hunks) > shown:
+ rest = [line for hunk in hunks[shown:] for line in hunk]
+ lines += ["", "{label}
", "", f"{fence}text"] - lines.extend(body) - lines.extend([fence, "", "", f"
"]
+ commits = [
+ f"[`{scope.sha[:7]}`]({_commit_url(scope.sha)})"
+ for scope in scopes[:shown]
+ if scope.label.startswith("Commit") and _commit_url(scope.sha)
+ ]
+ rules = list(
+ dict.fromkeys(
+ _rule_ref(check) for scope in scopes for check in _findings(scope, status)
+ )
+ )
+ links = []
+ if commits:
+ links.append(f"Commits: {' '.join(commits)}")
+ if rules:
+ links.append(f"Rules: {' · '.join(rules)}")
+ if links:
+ lines += ["", f"{' | '.join(links)}"]
+ return lines
+
+
+#: First line of the prompt "How to fix" offers for an AI coding agent.
+AGENT_PROMPT = "Fix these Commit Check findings without changing any code:"
+BRANCH_HELP = (
+ "**Branch name.** Rename the branch and push it. A pull request cannot "
+ "change its branch, so open a new one from the renamed branch:"
+)
+AUTHOR_HELP = (
+ "**Author.** Set your git identity (`git config user.name`, "
+ "`git config user.email`), then `git commit --amend --reset-author` "
+ "and force-push."
+)
+HOOK_HELP = (
+ f"**Next time.** The [pre-commit hook]({LOCAL_HOOK_URL}) runs the same "
+ "rules before you push."
+)
+
+
+def _agent_prompt(failing: list[ScopeResult]) -> list[str]:
+ """A prompt to paste into an AI coding agent, one line per failing scope."""
+ lines = [AGENT_PROMPT]
+ for scope in failing:
+ todo = list(dict.fromkeys(_instruction(scope, c) for c in scope.failures))
+ is_commit = bool(scope.sha) and scope.label.startswith("Commit")
+ label = f"Commit {scope.sha[:7]}" if is_commit else scope.label
+ lines.append(f"- {_target(scope, todo, label)}: {'; '.join(todo)}")
+ return lines
+
+def _is_last_commit(scope: ScopeResult) -> bool:
+ """Whether the scope is the newest commit: ``Commit 3/3`` or the HEAD one."""
+ position = re.fullmatch(r"Commit (\d+)/(\d+)", scope.label)
+ return not position or position.group(1) == position.group(2)
-def _markdown_code(value: str) -> str:
- """A code span that survives a GFM table cell.
- The value is user text (a commit subject, a branch name, an author) and
- commonly quotes code. A bare ```` `{value}` ```` breaks twice on that: a
- backtick inside closes the span early, and an unescaped ``|`` splits the
- cell, pushing the rule link into a fourth column the table drops. So the
- span uses one more backtick than the longest run inside the value, pads
- with a space when the value starts or ends with a backtick (GFM strips
- one on each side, so the padding is invisible), and escapes ``|``, which
- GFM honours even inside a code span when the span sits in a table cell.
+def _reword_commands(commits: list[ScopeResult]) -> list[str]:
+ """The git commands that reword the failing commits and push the result."""
+ if len(commits) == 1 and _is_last_commit(commits[0]):
+ reword = "git commit --amend"
+ else:
+ first = commits[0].sha[:7] or "and {len(hunks) - shown} more
"] + lines += ["", *_fenced("diff", rest), "", "", "
"]
-def _scope_value(scope: ScopeResult, max_len: int = 60) -> str:
- """First non-empty check value for a scope, trimmed to a single line.
+def _all_checks(results: list[ScopeResult]) -> list[str]:
+ """The folded listing of every scope, passed ones included.
- The value is the concrete thing that was checked (PR title, commit
- subject, branch name, author name/email) and reads naturally next to
- the scope label in the success details. The 60-character cap keeps the
- full line (prefix + value + parentheses) short enough to avoid wrapping
- in the fenced details block.
+ The same rows the step log prints (group name, one ✔/✖ row per scope with
+ the checked value), with the rule's reason, suggestion and fix under each
+ finding, so one expand answers both "what was checked" and "what failed
+ and why". The same block is used whether or not anything failed.
"""
- for check in scope.checks:
- value = check.get("value", "")
- if value:
- first_line = value.splitlines()[0].strip()
- if len(first_line) > max_len:
- return first_line[: max_len - 3] + "..."
- return first_line
- return ""
+ _failed, total = _check_counts(results)
+ unit = "check" if total == 1 else "checks"
+ label = f"All {total} {unit}" if total else "Details"
+ body = _fenced("text", _render_tree(results, include_docs=False))
+ return ["How to fix
"] + for part in parts: + lines.extend(["", *part]) + return [*lines, "", "", f"
"]
# ---------------------------------------------------------------------------
# Output specification
#
# The Markdown report shared by the job summary and the PR comment renders
-# as follows (values are filled from ScopeResult data):
+# as follows (values are filled from ScopeResult data; {label}
", "", *body, "", "+#
+#
-#
+# @@ Branch @@
+# - login-page
+# # Use / with an allowed type
# ```
#
-#
-#
-# _commit-check 2.13.1 · [Rules reference](https://commit-check.com/rules/)_
-#
-# A commit scope names its commit by short hash after the label, on every
-# surface; the label itself ("Commit 1/11") stays bare in the `result` output.
-#
-# Skipped (every rule declined to run — e.g. the author is in ignore_authors):
+# Commits: [`5584f46`](…) [`37d6def`](…) | Rules: [CC001](…) · [CC013](…) · [CC201](…)
#
-# ⊘ **All 5 checks skipped** — nothing was validated
+# Show all 5 checks
-# -# ```text -# Commit message -# ✔ PR title (feat: add login page) -# ✔ Commit 1/11 (d87faca) (feat: add user auth) -# Branch -# ✔ Branch (feature/add-login) -# Author -# ✔ Author name (Jane Doe) -# ✔ Author email (jane@example.com) +# ```diff +# @@ 5584f46 @@ +# - bad msg +# + fix: bad msg +# @@ 37d6def @@ +# feat: add OAuth providers +# - Co-authored-by: Claude
+# "
+# - Branch "login-page": Use / with an allowed type
# ```
#
-# The reason comes from commit-check; an older CLI sends none, and the line
-# reads "(skipped)" with no "Skipped because" sentence.
+# **Commit messages.** Reword them, then force-push:
#
-# A skipped scope deliberately carries no ✔ and no checked value: nothing was
-# examined, so there is no value to report and no pass to claim. When only some
-# scopes skip, the verdict reads "✅ **3 of 5 checks passed**, 2 skipped".
+# ```bash
+# git rebase -i 5584f46~1 # change "pick" to "reword" on the commits above
+# git push --force-with-lease
+# ```
#
-# Warning (the repository's config lists a rule under the top-level `warn`):
+# **Branch name.** Rename the branch and push it. …
#
-#
-# ## Commit Check
+# **Next time.** The [pre-commit hook](…) runs the same rules before you push.
#
-# ✅ **3 of 4 checks passed**, 1 warning
-#
-# | Scope | Checked value | Warnings |
-# |---|---|---|
-# | Branch | `jsmith/fix-x` | [CC201 branch](https://commit-check.com/rules/#cc201) |
+#
#
# How to fix
# -# Skipped because author dependabot[bot] is in ignore_authors. +# **With an AI coding agent.** Paste this prompt: # # ```text -# Commit message -# ⊘ PR title (skipped: author dependabot[bot] is in [commit].ignore_authors) -# ⊘ Commit 1/1 (skipped: author dependabot[bot] is in [commit].ignore_authors) -# Branch -# ⊘ Branch (skipped: author dependabot[bot] is in [branch].ignore_authors) -# Author -# ⊘ Author name (skipped: author dependabot[bot] is in [commit].ignore_authors) -# ⊘ Author email (skipped: author dependabot[bot] is in [commit].ignore_authors) +# Fix these Commit Check findings without changing any code: +# - Commit 5584f46: change "bad msg" to "fix: bad msg" +# - Commit 37d6def "feat: add OAuth providers": remove "Co-authored-by: Claude
-# /
-# Author
-# ✔ Author name (Jane Doe)
-# ✔ Author email (jane@example.com)
+# Suggest: Use / with an allowed type, or …
# ```
#
#
#
-# _commit-check 2.17.0 · [Rules reference](https://commit-check.com/rules/)_
+# Checked by [Commit Check](https://commit-check.com) 2.20.5 · [Rules](…) · [Check before you push](…)
#
-# A warned scope is reported exactly like a failing one — its own table row,
-# its own entry in the details block — except the marker is ⚠ rather than ✖,
-# it counts toward "passed" rather than "failed", and it never turns the
-# workflow red. A real failure elsewhere still fails the run; the verdict
-# then reads "❌ **N of M checks failed**, K warnings" and both tables appear.
+# Success: the heading reads "Show all 4 checks
+#All 5 checks
# # ```text -# Commit message -# ✔ PR title (feat: add login page) +# Pull request +# ✔ title feat: add login page +# Commits +# ✔ d87faca feat: add login page +# ✖ 5584f46 bad msg +# CC001 message +# value: bad msg +# The commit message should follow Conventional Commits. +# Fix: fix: bad msg +# ✖ 37d6def feat: add OAuth providers +# CC013 ai-attribution +# value: Claude Code +# AI attribution is forbidden in this project — detected: Claude Code +# Suggest: … +# Fix: feat: add OAuth providers # Branch -# ⚠ Branch (1 warning) +# ✖ branch login-page # CC201 branch -# value: jsmith/fix-x +# value: login-page # The branch should follow Conventional Branch. -# Suggest: Use
-# ():
-# ✖ Commit 3/11 (37d6def) (1 failure)
-# CC002 subject-capitalized
-# value: feat: add login page
-# Subject must start with a capital letter
-# Fix: feat: Add login page
-# Branch
-# ✔ Branch (feature/add-login)
-# ```
-#
-#
-#
-# _commit-check 2.13.1 · [Rules reference](https://commit-check.com/rules/)_
-#
-# The step log prints the same tree (plus a `Docs:` line per finding), then one
-# annotation per finding whose message is the tree's detail joined on `%0A`:
-#
-# ::error title=CC002 subject-capitalized::Commit 3/11 (37d6def): Subject must start with a capital letter%0Avalue: feat: add login page%0AFix: feat: Add login page
-# ✖ commit-check: 2 of 5 checks failed
+# …
+# ::endgroup::
+# ::error title=CC001 message::Commit 2/3 (5584f46): change "bad msg" to "fix: bad msg"%0AThe commit message should follow Conventional Commits.
+# ✖ Commit Check: 2 of 5 checks failed
+# How to fix: see the job summary. Check before you push: https://commit-check.com/…
#
# The verdict is a plain line, not an `::error`, so the run's error count is
# the number of findings.
#
# Notes:
-# - The table's Scope cell links to the commit (GITHUB_SERVER_URL, so it is
-# right on GitHub Enterprise Server too); the tree cannot, being a code block.
-# - `Fix:` is the corrected text the CLI proposes, when the correction is
-# mechanical (CC002 capitalisation, CC010 WIP marker, CC012 sign-off, ...).
-# When the CLI's `suggest` is just `Use "Show all 5 checks
-# -# ```text -# Commit message -# ✔ PR title (feat: add login page) -# ✖ Commit 2/11 (5584f46) (1 failure) +# ::group::✖ Commits: 2 of 3 failed +# ✔ d87faca feat: add login page +# ✖ 5584f46 bad msg # CC001 message -# value: bad msg -# The commit message should follow Conventional Commits. -# Suggest: Use\n" + f"{group_line('✔', 'passed', 'Commits: 2 passed')}
\n" + f"{group_line('✔', 'passed', 'Branch: passed')}\n" "\n" "
\n"
- "
\n"
@@ -1916,7 +2027,8 @@ def test_success_golden_output(self):
@pin_version
def test_failure_golden_output(self):
- """Pin the full failure report: failed row in the table, all in details."""
+ """Pin the full failure report: the diff of what failed, how to fix
+ it, and every scope in the listing."""
results = [
pass_scope("PR title", value="feat: add login page"),
fail_scope("Commit 2/2"),
@@ -1926,29 +2038,55 @@ def test_failure_golden_output(self):
self.assertEqual(
body,
f"{main.COMMENT_MARKER}\n"
- f"{main.REPORT_TITLE}\n"
+ f"{heading('✖', 'failed', 'failed')}\n"
"\n"
- "❌ **1 of 3 checks failed**\n"
+ f"{group_line('✔', 'passed', 'Pull request: passed')}Show all 4 checks
\n" + "All 4 checks
\n" "\n" "```text\n" "Pull request\n" - " ✔ PR title (feat: add login page)\n" - "Commit message\n" - " ✔ Commit 1/2 (feat: add user auth)\n" - " ✔ Commit 2/2 (fix: resolve timeout)\n" + " ✔ title feat: add login page\n" + "Commits\n" + " ✔ d87faca feat: add user auth\n" + " ✔ 5584f46 fix: resolve timeout\n" "Branch\n" - " ✔ Branch (feature/add-login)\n" + " ✔ branch feature/add-login\n" "```\n" "\n" "\n" + f"{group_line('✖', 'failed', 'Commits: failed')}
\n" + f"{group_line('✔', 'passed', 'Branch: passed')}\n" "\n" - "| Scope | Checked value | Failed checks |\n" - "|---|---|---|\n" - "| Commit 2/2 | `bad message` | " - "[CC001 message](https://commit-check.com/rules/#cc001) |\n" + "```diff\n" + "@@ Commit 2/2 @@\n" + "- bad message\n" + "# Use
\n"
- "(): \n'
+ "```\n"
+ "\n"
+ "**Commit messages.** Reword it, then force-push:\n"
+ "\n"
+ "```bash\n"
+ "git commit --amend\n"
+ "git push --force-with-lease\n"
+ "```\n"
+ "\n"
+ f"{HOOK_HELP}\n"
+ "\n"
+ "
\n"
+ "\n"
+ "Show all 3 checks
\n" + "How to fix
\n" + "\n" + "**With an AI coding agent.** Paste this prompt:\n" + "\n" + "```text\n" + f"{AGENT_PROMPT}\n" + '- Commit 2/2 "bad message": Use\n"
+ "(): \n"
"Branch\n"
- " ✔ Branch (feature/add-login)\n"
+ " ✔ branch feature/add-login\n"
"```\n"
"\n"
"
\n"
@@ -1958,8 +2096,9 @@ def test_failure_golden_output(self):
@pin_version
def test_failure_golden_output_with_commits(self):
- """Pin the report for a pull request's commits: the table row links
- to the commit, the tree names it, and a mechanical fix is shown."""
+ """Pin the report for a pull request's commits: each is named by its
+ short hash, linked under the diff, and a mechanical fix is a green
+ line."""
results = [
pass_scope("PR title", value="feat: Add login page"),
fail_scope("Commit 1/3", sha=SHA_A),
@@ -1971,42 +2110,74 @@ def test_failure_golden_output_with_commits(self):
env = {"GITHUB_REPOSITORY": "acme/widgets", "GITHUB_SERVER_URL": ""}
with patch.dict(os.environ, env):
body = main.render_report(results)
+ commits = "https://github.com/acme/widgets/commit"
self.assertEqual(
body,
f"{main.COMMENT_MARKER}\n"
- f"{main.REPORT_TITLE}\n"
+ f"{heading('✖', 'failed', 'failed')}\n"
+ "\n"
+ f"{group_line('✔', 'passed', 'Pull request: passed')}All 3 checks
\n" "\n" "```text\n" "Pull request\n" - " ✔ PR title (feat: add login page)\n" - "Commit message\n" - " ✖ Commit 2/2 (1 failure)\n" + " ✔ title feat: add login page\n" + "Commits\n" + " ✖ 2/2 bad message\n" " CC001 message\n" " value: bad message\n" " The commit message should follow Conventional Commits.\n" " Suggest: Use\n" + f"{group_line('✖', 'failed', 'Commits: 2 of 3 failed')}
\n" + f"{group_line('✔', 'passed', 'Branch: passed')}\n" "\n" - "❌ **2 of 5 checks failed**\n" + "```diff\n" + "@@ d87faca @@\n" + "- bad message\n" + "# Use
\n"
- "(): \n'
+ '- Commit 5584f46: change "feat: add login page" to '
+ '"feat: Add login page"\n'
+ "```\n"
+ "\n"
+ "**Commit messages.** Reword them, then force-push:\n"
+ "\n"
+ "```bash\n"
+ 'git rebase -i d87faca~1 # change "pick" to "reword" on the '
+ "commits above\n"
+ "git push --force-with-lease\n"
+ "```\n"
+ "\n"
+ f"{HOOK_HELP}\n"
+ "\n"
+ "
\n"
+ "\n"
+ "Show all 5 checks
\n" + "How to fix
\n" + "\n" + "**With an AI coding agent.** Paste this prompt:\n" + "\n" + "```text\n" + f"{AGENT_PROMPT}\n" + '- Commit d87faca "bad message": Use\n"
+ "(): \n"
- " ✖ Commit 2/3 (5584f46) (1 failure)\n"
+ " ✖ 5584f46 feat: add login page\n"
" CC002 subject-capitalized\n"
" value: feat: add login page\n"
" Subject must start with a capital letter\n"
" Fix: feat: Add login page\n"
- " ✔ Commit 3/3 (37d6def) (fix: Handle timeout)\n"
+ " ✔ 37d6def fix: Handle timeout\n"
"Branch\n"
- " ✔ Branch (feature/add-login)\n"
+ " ✔ branch feature/add-login\n"
"```\n"
"\n"
"
\n"
@@ -2014,7 +2185,7 @@ def test_failure_golden_output_with_commits(self):
f"{FOOTER}",
)
- def test_table_link_honours_the_server_url(self):
+ def test_commit_link_honours_the_server_url(self):
"""GitHub Enterprise Server is not github.com."""
env = {
"GITHUB_REPOSITORY": "acme/widgets",
@@ -2023,26 +2194,36 @@ def test_table_link_honours_the_server_url(self):
with patch.dict(os.environ, env):
body = main.render_report([fix_scope("Commit 1/1", sha=SHA_B)])
self.assertIn(
- f"| [Commit 1/1 (5584f46)](https://ghe.example.com/acme/widgets/commit/{SHA_B}) |",
+ f"Commits: [`5584f46`](https://ghe.example.com/acme/widgets/commit/{SHA_B}) ",
body,
)
- def test_table_has_no_link_without_a_repository_to_link_into(self):
+ def test_no_commit_link_without_a_repository_to_link_into(self):
with patch.dict(os.environ, {"GITHUB_REPOSITORY": ""}):
body = main.render_report([fix_scope("Commit 1/1", sha=SHA_B)])
- self.assertIn("| Commit 1/1 (5584f46) | `feat: add login page` |", body)
+ self.assertIn("@@ 5584f46 @@\n- feat: add login page\n", body)
+ self.assertIn("\nRules: [CC002](", body)
self.assertNotIn("/commit/", body)
def test_scopes_without_a_commit_are_never_linked(self):
with patch.dict(os.environ, {"GITHUB_REPOSITORY": "acme/widgets"}):
body = main.render_report([fail_scope("Branch")])
- self.assertIn("| Branch | `bad message` |", body)
+ self.assertIn("@@ Branch @@\n- bad message\n", body)
+ self.assertNotIn("/commit/", body)
+
+ def test_an_author_scope_is_named_by_its_label_not_its_commit(self):
+ """An author check runs on a commit, but the finding is the author."""
+ scope = fail_scope("Author email", sha=SHA_A)
+ with patch.dict(os.environ, {"GITHUB_REPOSITORY": "acme/widgets"}):
+ body = main.render_report([scope])
+ self.assertIn("@@ Author email @@\n", body)
+ self.assertIn(" ✖ email bad message\n", body)
self.assertNotIn("/commit/", body)
def test_skipped_scope_carries_the_sha_too(self):
scope = skip_scope("Commit 1/1")
scope.sha = SHA_A
- self.assertIn(" ⊘ Commit 1/1 (d87faca) (skipped)", main.render_report([scope]))
+ self.assertIn(" ⊘ d87faca skipped\n", main.render_report([scope]))
def test_a_check_is_a_thing_checked_not_a_rule_evaluation(self):
"""The total counts scopes, so it tracks the policy, not the PR size.
@@ -2060,15 +2241,18 @@ def test_a_check_is_a_thing_checked_not_a_rule_evaluation(self):
make_check("subject_min_length", status="fail", rule_id="CC005"),
],
)
- body = main.render_report([pass_scope("Branch"), two_failures])
- self.assertIn("❌ **1 of 2 checks failed**", body)
- self.assertIn("All 5 checks
\n" "\n" "```text\n" "Pull request\n" - " ✔ PR title (feat: Add login page)\n" - "Commit message\n" - " ✖ Commit 1/3 (d87faca) (1 failure)\n" + " ✔ title feat: Add login page\n" + "Commits\n" + " ✖ d87faca bad message\n" " CC001 message\n" " value: bad message\n" " The commit message should follow Conventional Commits.\n" " Suggest: Use", body)
- self.assertIn("(): \n"
+ "```\n",
body,
)
- self.assertIn("
Show all 1 check
", body) + self.assertIn("All 1 check
", body) self.assertIn("```text", body) - self.assertIn("Branch", body) - self.assertIn(" ✔ Branch (main)", body) + self.assertIn("\nBranch\n ✔ branch main\n", body) + self.assertNotIn("```diff", body) + self.assertNotIn("How to fix", body) def test_all_pass_groups_scopes_like_step_log(self): results = [ @@ -2111,42 +2304,61 @@ def test_all_pass_groups_scopes_like_step_log(self): pass_scope("Author email", value="jane@example.com"), ] body = main.render_job_summary(results) - # Group headers in the details block mirror the step log ordering. - self.assertLess(body.index("Commit message"), body.index("Branch")) - self.assertLess(body.index("Branch"), body.index("Author")) - self.assertIn(" ✔ PR title (feat: add login page)", body) - self.assertIn(" ✔ Branch (feature/pr-12)", body) - self.assertIn(" ✔ Author email (jane@example.com)", body) + listing = body.split("```text\n")[1] + # Group headers in the listing mirror the step log ordering. + self.assertLess(listing.index("Commits\n"), listing.index("Branch\n")) + self.assertLess(listing.index("Branch\n"), listing.index("Author\n")) + self.assertIn(" ✔ title feat: add login page\n", listing) + self.assertIn(" ✔ branch feature/pr-12\n", listing) + self.assertIn(" ✔ email jane@example.com\n", listing) + self.assertIn("> Author: 2 passed", body) def test_all_pass_truncates_long_values(self): long_value = "x" * 200 body = main.render_job_summary([pass_scope("Commit 1/1", value=long_value)]) - self.assertIn(f" ✔ Commit 1/1 ({'x' * 57}...)", body) + self.assertIn(f" ✔ 1/1 {'x' * 57}...\n", body) def test_all_pass_without_value_shows_plain_label(self): body = main.render_job_summary([pass_scope("Branch")]) - self.assertIn(" ✔ Branch", body) - self.assertNotIn(" ✔ Branch (", body) + self.assertIn(" ✔ branch\n", body) - def test_failure_renders_table_with_rule_links(self): + def test_failure_shows_the_diff_and_the_rule_links(self): body = main.render_job_summary([fail_scope("Commit 1/1")]) - self.assertIn(main.REPORT_TITLE, body) - self.assertIn("❌ **1 of 1 check failed**", body) - self.assertIn("| Scope | Checked value | Failed checks |", body) + self.assertIn("Commit Check failed\n", body) + self.assertIn("> Commits: failed\n", body) self.assertIn( - "| Commit 1/1 | `bad message` | " - "[CC001 message](https://commit-check.com/rules/#cc001) |", + "```diff\n" + "@@ Commit 1/1 @@\n" + "- bad message\n" + "# Use", body)
- self.assertIn("(): ", body)
- self.assertIn("[Rules reference](https://commit-check.com/rules/)", body)
+ self.assertIn("[Rules](https://commit-check.com/rules/)", body)
+
+ def test_a_failing_value_is_printed_in_full(self):
+ """It is the one value the reader has to act on; the cap is for the
+ rows that passed."""
+ subject = "feat: " + "x" * 100
+ scope = main.ScopeResult(
+ label="Commit 1/1",
+ checks=[
+ make_check(
+ "subject_max_length", status="fail", rule_id="CC004", value=subject
+ )
+ ],
+ )
+ body = main.render_report([scope])
+ self.assertIn(f"- {subject}\n", body)
+ self.assertIn(f" ✖ 1/1 {subject}\n", body)
def test_failure_details_show_all_scopes_and_values(self):
results = [
@@ -2154,16 +2366,326 @@ def test_failure_details_show_all_scopes_and_values(self):
pass_scope("Commit 2/2", value="fix: resolve timeout"),
]
body = main.render_job_summary(results)
- self.assertIn("✔ Commit 2/2 (fix: resolve timeout)", body)
- self.assertIn("✖ Commit 1/2 (1 failure)", body)
+ self.assertIn(" ✔ 2/2 fix: resolve timeout\n", body)
+ self.assertIn(" ✖ 1/2 bad message\n", body)
- def test_pass_scope_renders_checkmark_without_value(self):
+ def test_passing_scopes_stay_out_of_the_diff(self):
body = main.render_job_summary([pass_scope("Branch"), fail_scope("Commit 1/1")])
- # Pass scopes stay out of the table; the details block carries them.
- table = body.split("
Show all 1 check
", body) - self.assertIn("```text", body) - self.assertIn("✖ Commit 1/1 (1 failure)", body) + self.assertIn( + "Rules: [CC001](https://commit-check.com/rules/#cc001)", body + ) + self.assertIn("All 1 check
", body) + self.assertIn(" ✖ 1/1 bad message\n", body) self.assertIn(" CC001 message", body) - # The failing value appears in full; the table truncates at 60. self.assertIn(" value: bad message", body) self.assertIn(" Suggest: Use")[0]
- self.assertNotIn("| Branch |", table)
- self.assertIn("| Commit 1/1 | `bad message` |", table)
- self.assertIn("✔ Branch", body)
+ diff = body.split("```diff\n")[1].split("```")[0]
+ self.assertEqual(diff.count("@@ "), 1)
+ self.assertIn("@@ Commit 1/1 @@", diff)
+ self.assertIn(" ✔ branch\n", body)
+
+
+def ai_scope(label: str = "Commit 1/1", sha: str = SHA_A) -> main.ScopeResult:
+ """A commit that credits an AI co-author, under ``ai_attribution = "forbid"``:
+ the rule's value is the tool, and its fix the message without the trailer."""
+ return main.ScopeResult(
+ label=label,
+ sha=sha,
+ text=(
+ "feat: add OAuth providers\n\n"
+ "Co-authored-by: Claude \n"
+ ),
+ checks=[
+ make_check(
+ "ai_attribution",
+ status="fail",
+ rule_id="CC013",
+ value="Claude Code",
+ error="AI attribution is forbidden in this project",
+ suggest="Remove the AI trailer lines and re-commit.",
+ fix="feat: add OAuth providers",
+ docs_url="https://commit-check.com/rules/#cc013",
+ )
+ ],
+ )
+
+
+class TestLineEdits(unittest.TestCase):
+ """What a rule's fix removes and adds, which the diff shows red and green."""
+
+ def test_a_subject_rule_is_compared_with_the_subject_alone(self):
+ """Its fix is the subject, so the body is not something it removed."""
+ scope = fix_scope()
+ scope.text = "feat: add login page\n\nThe body stays.\n"
+ self.assertEqual(
+ main._line_edits(scope, scope.checks[0]),
+ (["feat: add login page"], ["feat: Add login page"]),
+ )
+
+ def test_a_whole_message_fix_is_compared_with_the_whole_message(self):
+ message = "feat: add login page\n\nThe body stays."
+ scope = main.ScopeResult(
+ label="Commit 1/1",
+ text=message + "\n",
+ checks=[
+ make_check(
+ "require_signed_off_by",
+ status="fail",
+ rule_id="CC012",
+ value=message,
+ fix=f"{message}\n\nSigned-off-by: Jane Doe ",
+ )
+ ],
+ )
+ self.assertEqual(
+ main._line_edits(scope, scope.checks[0]),
+ ([], ["Signed-off-by: Jane Doe "]),
+ )
+
+ def test_an_ai_rule_is_compared_with_the_message_not_with_its_value(self):
+ """The value is the tool's name; the fix drops the trailer."""
+ scope = ai_scope()
+ self.assertEqual(
+ main._line_edits(scope, scope.checks[0]),
+ (["Co-authored-by: Claude "], []),
+ )
+
+ def test_an_ai_rule_without_the_message_has_no_line_to_name(self):
+ scope = ai_scope()
+ scope.text = ""
+ self.assertEqual(main._line_edits(scope, scope.checks[0]), ([], []))
+ self.assertEqual(
+ main._instruction(scope, scope.checks[0]),
+ "Remove the AI trailer lines and re-commit.",
+ )
+
+ def test_the_message_rule_supplies_the_text_outside_a_pull_request(self):
+ """Nothing is piped in for the HEAD commit, but the message rule's
+ value is the whole message."""
+ scope = ai_scope()
+ scope.checks.insert(0, make_check("message", value=scope.text.strip()))
+ scope.text = ""
+ self.assertEqual(
+ main._line_edits(scope, scope.checks[1]),
+ (["Co-authored-by: Claude "], []),
+ )
+ self.assertEqual(main._first_value(scope), "feat: add OAuth providers")
+
+ def test_a_rule_without_a_fix_has_no_edits(self):
+ scope = fail_scope()
+ self.assertEqual(main._line_edits(scope, scope.checks[0]), ([], []))
+
+ def test_a_fix_with_nothing_to_compare_it_with_has_no_edits(self):
+ scope = main.ScopeResult(
+ label="Branch", checks=[make_check("branch", status="fail", fix="x")]
+ )
+ self.assertEqual(main._line_edits(scope, scope.checks[0]), ([], []))
+
+ def test_recorded_edits_win_over_the_fix(self):
+ """A description is too long to carry as a fix, so its line changes
+ are recorded when it is checked."""
+ scope = fail_scope("PR description")
+ scope.edits["CC001"] = (["Generated with Claude Code"], [])
+ self.assertEqual(
+ main._line_edits(scope, scope.checks[0]),
+ (["Generated with Claude Code"], []),
+ )
+
+ def test_instruction_names_each_kind_of_change(self):
+ scope = fail_scope()
+ check = scope.checks[0]
+ cases = [
+ ((["a"], ["b"]), 'change "a" to "b"'),
+ ((["a", "b"], []), 'remove "a"; "b"'),
+ (([], ["c"]), 'add "c"'),
+ ((["a", "b"], ["c"]), 'remove "a"; "b", and add "c"'),
+ ]
+ for edits, expected in cases:
+ with self.subTest(expected=expected):
+ scope.edits["CC001"] = edits
+ self.assertEqual(main._instruction(scope, check), expected)
+
+ def test_advice_drops_what_only_the_rule_page_needs(self):
+ cases = {
+ "Use (): , where is one of: feat, fix": (
+ "Use (): "
+ ),
+ "Use / with an allowed type, or add the branch "
+ "to allow_branch_names in config": (
+ "Use / with an allowed type"
+ ),
+ 'In the description, remove "x"': 'remove "x"',
+ "Add a body\nexplaining the change": "Add a body",
+ }
+ for suggest, expected in cases.items():
+ with self.subTest(suggest=suggest):
+ check = make_check("message", status="fail", suggest=suggest)
+ self.assertEqual(main._advice(check), expected)
+
+ def test_advice_falls_back_to_the_error_without_its_link(self):
+ check = make_check(
+ "message",
+ status="fail",
+ error="Follow Conventional Commits. See https://www.conventionalcommits.org",
+ )
+ self.assertEqual(main._advice(check), "Follow Conventional Commits.")
+
+
+class TestDiffBlock(unittest.TestCase):
+ def _hunks(self, results, status="fail"):
+ lines = main._diff_block(results, status)
+ return lines[1 : lines.index("```")]
+
+ def test_a_fix_on_another_line_leaves_the_subject_as_context(self):
+ self.assertEqual(
+ self._hunks([ai_scope()]),
+ [
+ "@@ d87faca @@",
+ " feat: add OAuth providers",
+ "- Co-authored-by: Claude ",
+ ],
+ )
+
+ def test_advice_is_about_the_value_so_the_value_is_red(self):
+ scope = fail_scope("Commit 1/1", sha=SHA_A)
+ scope.checks.append(
+ make_check(
+ "subject_min_length",
+ status="fail",
+ rule_id="CC005",
+ value="bad message",
+ suggest="Write a subject of at least 20 characters",
+ )
+ )
+ self.assertEqual(
+ self._hunks([scope]),
+ [
+ "@@ d87faca @@",
+ "- bad message",
+ "# Use (): ",
+ "# Write a subject of at least 20 characters",
+ ],
+ )
+
+ def test_two_rules_that_ask_for_the_same_line_show_it_once(self):
+ scope = ai_scope()
+ scope.checks.append({**scope.checks[0], "rule_id": "CC015"})
+ self.assertEqual(len(self._hunks([scope])), 3)
+
+ def test_a_description_is_named_by_its_label_and_shows_only_the_lines(self):
+ scope = fail_scope("PR description")
+ scope.text = "Adds login.\n\nGenerated with Claude Code\n"
+ scope.edits["CC001"] = (["Generated with Claude Code"], [])
+ self.assertEqual(
+ self._hunks([scope]),
+ ["@@ PR description @@", "- Generated with Claude Code"],
+ )
+
+ def test_a_long_list_is_cut_and_the_rest_folded(self):
+ results = [fail_scope(f"Commit {n}/8", sha=f"{n}" * 40) for n in range(1, 9)]
+ with patch.dict(os.environ, {"GITHUB_REPOSITORY": "acme/widgets"}):
+ lines = main._diff_block(results)
+ body = "\n".join(lines)
+ shown, folded = body.split("
")
+ self.assertEqual(shown.count("@@ "), main.MAX_HUNKS)
+ self.assertIn("
\n" + f"{group_line('⊘', 'skipped', 'Commits: skipped')}\n" "\n" "
\n", body) + # Two scopes in the group, one of which ran: never "2 passed". + self.assertIn("> Pull request: 1 passed, 1 skipped\n", body) def test_failure_still_wins_over_skips(self): results = [fail_scope("Commit 1/1"), skip_scope("PR title")] body = main.render_report(results) - self.assertIn("❌ **1 of 2 checks failed**", body) + self.assertIn("Commit Check failed\n", body) + self.assertIn("> Commits: failed
\n", body) + self.assertIn("> Pull request: skipped\n", body) def test_skipped_scope_reports_no_checked_value(self): """The ⊘ line carries no value, because nothing was examined.""" body = main.render_report([skip_scope("PR title")]) - self.assertIn(" ⊘ PR title (skipped)", body) + self.assertIn(" ⊘ title skipped\n", body) def test_skip_reason_is_shown_on_the_scope_and_under_the_verdict(self): """A skip says why, so a deliberate bypass reads as one.""" @@ -3098,20 +3640,21 @@ def test_skip_reason_is_shown_on_the_scope_and_under_the_verdict(self): skip_scope("Branch", reason=branch), ] body = main.render_report(results) + self.assertIn("Commit Check skipped\n\nNothing was validated.\n", body) self.assertIn( - "⊘ **All 3 checks skipped** — nothing was validated\n\n" + "> Branch: skipped\n\n" "Skipped because author dependabot[bot] is in ignore_authors.\n", body, ) - self.assertIn(f" ⊘ PR title (skipped: {commit})", body) - self.assertIn(f" ⊘ Branch (skipped: {branch})", body) + self.assertIn(f" ⊘ title skipped: {commit}\n", body) + self.assertIn(f" ⊘ branch skipped: {branch}\n", body) def test_skip_reason_follows_a_partial_skip_verdict(self): reason = "author dependabot[bot] is in [branch].ignore_authors" results = [pass_scope("PR title"), skip_scope("Branch", reason=reason)] body = main.render_report(results) self.assertIn( - "✅ **1 of 2 checks passed**, 1 skipped\n\n" + "> Branch: skipped\n\n" "Skipped because author dependabot[bot] is in ignore_authors.\n", body, ) @@ -3120,7 +3663,10 @@ def test_skip_reason_reaches_the_step_log(self): reason = "author dependabot[bot] is in [commit].ignore_authors" with patch("sys.stdout", new_callable=io.StringIO) as out: main.render_step_log([skip_scope("PR title", reason=reason)]) - self.assertIn(f" ⊘ PR title (skipped: {reason})", out.getvalue()) + self.assertIn("::group::⊘ Pull request: skipped\n", out.getvalue()) + self.assertIn( + f" ⊘ title skipped: {reason}\n", ANSI_CODES.sub("", out.getvalue()) + ) self.assertIn( "Skipped because author dependabot[bot] is in ignore_authors.", out.getvalue(), @@ -3142,7 +3688,7 @@ def test_a_passing_scope_has_no_skip_reason(self): def test_older_engine_without_skip_is_unaffected(self): """Back-compat: engines that only emit pass/fail render as before.""" results = [pass_scope("Branch", value="main")] - self.assertIn("✅ **All 1 check passed**", main.render_report(results)) + self.assertIn("Commit Check passed\n", main.render_report(results)) def warn_scope(label: str = "Branch", value: str = "jsmith/fix-x") -> main.ScopeResult: @@ -3183,9 +3729,9 @@ def test_a_real_failure_outranks_a_warning_in_the_same_scope(self): def test_a_warning_survives_reporting_on_a_scope_that_also_fails(self): """The scope's status names its worst outcome, but both findings - still belong on every surface: the count, both tables, and the - details block. Filtering any of those on ``scope.status == "warn"`` - drops this scope's warning the moment its sibling rule fails. + still belong on every surface: the count, both diffs, and the + listing. Filtering any of those on ``scope.status == "warn"`` drops + this scope's warning the moment its sibling rule fails. """ mixed = main.ScopeResult( label="Branch", @@ -3212,15 +3758,33 @@ def test_a_warning_survives_reporting_on_a_scope_that_also_fails(self): self.assertEqual(main._warn_count([mixed]), 1) body = main.render_report([mixed]) - self.assertIn("❌ **1 of 1 check failed**, 1 warning", body) - self.assertIn("| Scope | Checked value | Failed checks |", body) - self.assertIn("[CC202 merge-base](https://commit-check.com/rules/#cc202)", body) - self.assertIn("| Scope | Checked value | Warnings |", body) - self.assertIn("[CC201 branch](https://commit-check.com/rules/#cc201)", body) - self.assertIn(" ✖ Branch (1 failure)", body) - self.assertIn(" CC202 merge-base", body) - self.assertIn(" ⚠ Branch (1 warning)", body) - self.assertIn(" CC201 branch", body) + self.assertIn("Commit Check failed\n", body) + failed, warned = body.split("**Warnings**") + self.assertIn( + "```diff\n" + "@@ Branch @@\n" + "- jsmith/fix-x\n" + "# Current branch is not rebased onto main.\n" + "```\n" + "\n" + "Rules: [CC202](https://commit-check.com/rules/#cc202)\n", + failed, + ) + self.assertIn( + "```diff\n" + "@@ Branch @@\n" + "- jsmith/fix-x\n" + "# Use/\n"
+ "```\n"
+ "\n"
+ "Rules: [CC201](https://commit-check.com/rules/#cc201)\n",
+ warned,
+ )
+ # One row, marked with the worse of the two, and both findings under
+ # it; the warning says that it is one.
+ self.assertIn(" ✖ branch jsmith/fix-x\n CC202 merge-base\n", body)
+ self.assertIn("\n CC201 branch (warning)\n", body)
+ self.assertNotIn("⚠ branch", body)
def test_step_log_and_report_agree_on_a_mixed_scope(self):
"""The step log's ::warning annotation must not be the only surface
@@ -3297,24 +3861,30 @@ def test_warning_only_golden_output(self):
self.assertEqual(
body,
f"{main.COMMENT_MARKER}\n"
- f"{main.REPORT_TITLE}\n"
+ f"{heading('⚠', 'warned', 'passed with warnings')}\n"
"\n"
- "✅ **1 of 2 checks passed**, 1 warning\n"
+ f"{group_line('✔', 'passed', 'Pull request: passed')}
\n" + f"{group_line('⚠', 'warned', 'Branch: 1 warning')}\n" "\n" - "| Scope | Checked value | Warnings |\n" - "|---|---|---|\n" - "| Branch | `jsmith/fix-x` | " - "[CC201 branch](https://commit-check.com/rules/#cc201) |\n" + "**Warnings**\n" + "\n" + "```diff\n" + "@@ Branch @@\n" + "- jsmith/fix-x\n" + "# Use/\n"
+ "```\n"
+ "\n"
+ "Rules: [CC201](https://commit-check.com/rules/#cc201)\n"
"\n"
"
and 3 more
\n\n```diff\n@@ 6666666 @@", folded) + self.assertEqual(folded.count("@@ "), 3) + # Only the commits in view are linked; the rule is named once. + links = lines[-1] + self.assertEqual(links.count("/commit/"), main.MAX_HUNKS) + self.assertNotIn("6666666", links) + self.assertEqual(links.count("[CC001]"), 1) + + def test_one_or_two_leftovers_are_not_worth_a_fold(self): + results = [fail_scope(f"Commit {n}/7", sha=f"{n}" * 40) for n in range(1, 8)] + body = "\n".join(main._diff_block(results)) + self.assertEqual(body.count("@@ "), 7) + self.assertNotIn("", body)
+
+ def test_warnings_get_the_same_block(self):
+ self.assertEqual(
+ self._hunks([warn_scope(), fail_scope()], "warn"),
+ ["@@ Branch @@", "- jsmith/fix-x", "# Use /"],
+ )
+
+
+class TestHowToFix(unittest.TestCase):
+ def _body(self, results):
+ return "\n".join(main._how_to_fix(results))
+
+ def test_nothing_failed_nothing_to_fix(self):
+ self.assertEqual(main._how_to_fix([pass_scope(), warn_scope()]), [])
+
+ def test_the_prompt_lists_every_failing_scope_with_its_instruction(self):
+ results = [
+ fail_scope("PR title"),
+ ai_scope("Commit 1/2"),
+ fix_scope("Commit 2/2"),
+ pass_scope("Branch"),
+ ]
+ self.assertIn(
+ "```text\n"
+ f"{AGENT_PROMPT}\n"
+ '- PR title "bad message": Use (): \n'
+ '- Commit d87faca "feat: add OAuth providers": remove '
+ '"Co-authored-by: Claude "\n'
+ '- Commit 5584f46: change "feat: add login page" to '
+ '"feat: Add login page"\n'
+ "```\n",
+ self._body(results),
+ )
+
+ def test_only_the_last_commit_failing_is_an_amend(self):
+ for scope in (
+ fail_scope("Commit 3/3", sha=SHA_A),
+ fail_scope("Commit message"),
+ ):
+ with self.subTest(label=scope.label):
+ body = self._body([scope])
+ self.assertIn("Reword it, then force-push:", body)
+ self.assertIn("```bash\ngit commit --amend\ngit push --force", body)
+ self.assertNotIn("rebase", body)
+
+ def test_an_earlier_commit_needs_a_rebase_from_its_parent(self):
+ body = self._body(
+ [fail_scope("Commit 2/3", sha=SHA_B), pass_scope("Commit 3/3")]
+ )
+ self.assertIn("```bash\ngit rebase -i 5584f46~1 # change", body)
+
+ def test_a_commit_without_a_hash_gets_a_placeholder(self):
+ body = self._body([fail_scope("Commit 1/3"), fail_scope("Commit 2/3")])
+ self.assertIn("Reword them, then force-push:", body)
+ self.assertIn("git rebase -i ~1 # change", body)
+
+ def test_each_kind_of_failure_gets_its_own_paragraph(self):
+ body = self._body(
+ [
+ fail_scope("PR title"),
+ fail_scope("PR description"),
+ fail_scope("Author email", sha=SHA_A),
+ fail_scope("Branch"),
+ ]
+ )
+ self.assertNotIn("**Commit messages.**", body)
+ self.assertIn("**Author.** Set your git identity", body)
+ self.assertIn(
+ "**Branch name.** Rename the branch and push it. A pull request cannot "
+ "change its branch, so open a new one from the renamed branch:\n\n"
+ "```bash\ngit branch -m /\ngit push -u origin HEAD\n```",
+ body,
+ )
+ self.assertIn(
+ "**Pull request title and description.** Edit them on the pull "
+ "request page.",
+ body,
+ )
+ self.assertTrue(body.endswith(f"{HOOK_HELP}\n\n
"))
+
+ def test_the_branch_command_uses_the_corrected_name_when_there_is_one(self):
+ scope = fail_scope("Branch")
+ scope.checks[0].update(value="Feature/Add-Login", fix="feature/Add-Login")
+ self.assertIn(
+ "```bash\ngit branch -m feature/Add-Login\ngit push -u origin HEAD\n```",
+ self._body([scope]),
+ )
+
+ def test_one_pull_request_field_is_it_not_them(self):
+ body = self._body([fail_scope("PR title")])
+ self.assertIn("**Pull request title.** Edit it on the pull request page.", body)
+ self.assertNotIn("**Branch name.**", body)
+ self.assertNotIn("**Author.**", body)
class TestRenderPrComment(unittest.TestCase):
@@ -2172,34 +2694,41 @@ def test_all_pass_matches_job_summary(self):
summary = main.render_job_summary([pass_scope("Branch")])
self.assertEqual(comment, summary)
self.assertTrue(comment.startswith(main.COMMENT_MARKER))
- self.assertIn("✅ **All 1 check passed**", comment)
+ self.assertIn("Commit Check passed\n", comment)
def test_failure_matches_job_summary(self):
comment = main.render_pr_comment([fail_scope("Commit 1/1")])
summary = main.render_job_summary([fail_scope("Commit 1/1")])
self.assertEqual(comment, summary)
self.assertTrue(comment.startswith(main.COMMENT_MARKER))
- self.assertIn("❌ **1 of 1 check failed**", comment)
- self.assertIn("| Scope | Checked value | Failed checks |", comment)
+ self.assertIn("Commit Check failed\n", comment)
+ self.assertIn("```diff\n@@ Commit 1/1 @@\n", comment)
class TestUnparsableScopeInTheReport(unittest.TestCase):
"""A CLI response that is not JSON is a failure with no rule to link."""
- def test_the_table_row_points_at_the_details(self):
+ def test_the_diff_points_at_the_step_log(self):
scope = main.ScopeResult(label="Branch", raw_text="Error: cchk.toml: oops")
body = main.render_report([scope])
- self.assertIn("❌ **1 of 1 check failed**", body)
+ self.assertIn("Commit Check failed\n", body)
self.assertIn(
- "| Branch | — | _output could not be parsed — see details_ |", body
+ "```diff\n"
+ "@@ Branch @@\n"
+ "# the output could not be parsed, see the step log\n"
+ "```\n",
+ body,
)
- # The details block is where the raw output went.
- self.assertIn(" ✖ Branch\n Error: cchk.toml: oops\n", body)
+ # No rule spoke, so there is none to link and nothing to instruct.
+ self.assertNotIn("Rules", body)
+ self.assertNotIn("How to fix", body)
+ # The listing is where the raw output went.
+ self.assertIn(" ✖ branch\n Error: cchk.toml: oops\n", body)
- def test_it_never_reaches_the_warnings_table(self):
+ def test_it_never_reaches_the_warnings(self):
scope = main.ScopeResult(label="Branch", raw_text="garbage")
- table = main._markdown_table([scope], "warn", "Warnings")
- self.assertEqual(table.splitlines()[2:], [])
+ self.assertNotIn("**Warnings**", main.render_report([scope]))
+ self.assertEqual(main._diff_block([scope], "warn")[:2], ["```diff", "```"])
class TestReportFooter(unittest.TestCase):
@@ -2208,8 +2737,9 @@ def test_names_the_installed_commit_check(self):
self.assertEqual(main._commit_check_version(), "9.8.7")
self.assertEqual(
main._report_footer(),
- "_commit-check 9.8.7 · "
- "[Rules reference](https://commit-check.com/rules/)_",
+ "Checked by [Commit Check](https://commit-check.com) 9.8.7 · "
+ "[Rules](https://commit-check.com/rules/) · "
+ f"[Check before you push]({HOOK_GUIDE})",
)
version.assert_called_with("commit-check")
@@ -2220,7 +2750,9 @@ def test_without_a_version_it_still_links_the_rules(self):
self.assertEqual(main._commit_check_version(), "")
self.assertEqual(
main._report_footer(),
- "_[Rules reference](https://commit-check.com/rules/)_",
+ "Checked by [Commit Check](https://commit-check.com) · "
+ "[Rules](https://commit-check.com/rules/) · "
+ f"[Check before you push]({HOOK_GUIDE})",
)
@@ -2240,7 +2772,7 @@ def test_success_writes_policy_report(self):
self.assertEqual(rc, 0)
with open(summary_path, encoding="utf-8") as file_obj:
content = file_obj.read()
- self.assertIn("✅ **All 1 check passed**", content)
+ self.assertIn("Commit Check passed\n", content)
def test_failure_returns_nonzero(self):
summary_path = os.path.join(tempfile.mkdtemp(), "summary.txt")
@@ -2252,8 +2784,8 @@ def test_failure_returns_nonzero(self):
self.assertEqual(rc, 1)
with open(summary_path, encoding="utf-8") as file_obj:
content = file_obj.read()
- self.assertIn("| Scope | Checked value | Failed checks |", content)
- self.assertIn("❌", content)
+ self.assertIn("Commit Check failed\n", content)
+ self.assertIn("```diff\n@@ Commit 1/1 @@\n- bad message\n", content)
def read_github_output(path: str) -> dict[str, str]:
@@ -2443,7 +2975,7 @@ def test_fork_pr_warning_names_only_the_surfaces_that_exist(self):
rc = main.add_pr_comments([pass_scope()])
self.assertEqual(rc, 0)
printed = mock_print.call_args[0][0]
- self.assertIn("annotations on the Files changed tab", printed)
+ self.assertIn("annotations on the run page", printed)
self.assertNotIn("job's summary", printed)
self.assertFalse(os.path.exists(summary_path))
@@ -2505,7 +3037,7 @@ def test_creates_comment_with_rendered_body(self):
self.assertEqual(mock_pull_request.create_comment.call_count, 1)
body = mock_pull_request.create_comment.call_args[1]["body"]
self.assertTrue(body.startswith(main.COMMENT_MARKER))
- self.assertIn("| Scope | Checked value | Failed checks |", body)
+ self.assertIn("```diff\n@@ Commit 1/1 @@\n- bad message\n", body)
def test_updates_existing_comment_when_changed(self):
# A report from an earlier version: no marker, and posted by the bot,
@@ -2951,7 +3483,9 @@ def test_running_the_script_runs_main(self):
mock_run.assert_not_called()
self.assertIn("::debug::MESSAGE=false\n", out.getvalue())
self.assertTrue(
- out.getvalue().endswith("✔ commit-check: all checks passed\n"),
+ ANSI_CODES.sub("", out.getvalue()).endswith(
+ "✔ Commit Check: all checks passed\n"
+ ),
out.getvalue(),
)
@@ -3053,18 +3587,21 @@ def test_all_skipped_golden_output(self):
self.assertEqual(
body,
f"{main.COMMENT_MARKER}\n"
- f"{main.REPORT_TITLE}\n"
+ f"{heading('⊘', 'skipped', 'skipped')}\n"
"\n"
- "⊘ **All 2 checks skipped** — nothing was validated\n"
+ "Nothing was validated.\n"
+ "\n"
+ f"{group_line('⊘', 'skipped', 'Pull request: skipped')}\n" + f"{group_line('⊘', 'skipped', 'Commits: skipped')}\n" "\n" "
\n"
- "
\n"
@@ -3074,19 +3611,24 @@ def test_all_skipped_golden_output(self):
def test_partial_skip_does_not_claim_all_passed(self):
results = [pass_scope("Branch", value="main"), skip_scope("PR title")]
+ results.append(pass_scope("PR description"))
body = main.render_report(results)
- self.assertIn("✅ **1 of 2 checks passed**, 1 skipped", body)
- self.assertNotIn("All 2 checks passed", body)
+ self.assertIn("Commit Check passed\n", body)
+ self.assertIn("> Branch: passedShow all 2 checks
\n" + "All 2 checks
\n" "\n" "```text\n" "Pull request\n" - " ⊘ PR title (skipped)\n" - "Commit message\n" - " ⊘ Commit 1/1 (skipped)\n" + " ⊘ title skipped\n" + "Commits\n" + " ⊘ 1/1 skipped\n" "```\n" "\n" "\n", body) + # Two scopes in the group, one of which ran: never "2 passed". + self.assertIn("> Pull request: 1 passed, 1 skipped\n", body) def test_failure_still_wins_over_skips(self): results = [fail_scope("Commit 1/1"), skip_scope("PR title")] body = main.render_report(results) - self.assertIn("❌ **1 of 2 checks failed**", body) + self.assertIn("Commit Check failed\n", body) + self.assertIn("> Commits: failed
\n", body) + self.assertIn("> Pull request: skipped\n", body) def test_skipped_scope_reports_no_checked_value(self): """The ⊘ line carries no value, because nothing was examined.""" body = main.render_report([skip_scope("PR title")]) - self.assertIn(" ⊘ PR title (skipped)", body) + self.assertIn(" ⊘ title skipped\n", body) def test_skip_reason_is_shown_on_the_scope_and_under_the_verdict(self): """A skip says why, so a deliberate bypass reads as one.""" @@ -3098,20 +3640,21 @@ def test_skip_reason_is_shown_on_the_scope_and_under_the_verdict(self): skip_scope("Branch", reason=branch), ] body = main.render_report(results) + self.assertIn("Commit Check skipped\n\nNothing was validated.\n", body) self.assertIn( - "⊘ **All 3 checks skipped** — nothing was validated\n\n" + "> Branch: skipped\n\n" "Skipped because author dependabot[bot] is in ignore_authors.\n", body, ) - self.assertIn(f" ⊘ PR title (skipped: {commit})", body) - self.assertIn(f" ⊘ Branch (skipped: {branch})", body) + self.assertIn(f" ⊘ title skipped: {commit}\n", body) + self.assertIn(f" ⊘ branch skipped: {branch}\n", body) def test_skip_reason_follows_a_partial_skip_verdict(self): reason = "author dependabot[bot] is in [branch].ignore_authors" results = [pass_scope("PR title"), skip_scope("Branch", reason=reason)] body = main.render_report(results) self.assertIn( - "✅ **1 of 2 checks passed**, 1 skipped\n\n" + "> Branch: skipped\n\n" "Skipped because author dependabot[bot] is in ignore_authors.\n", body, ) @@ -3120,7 +3663,10 @@ def test_skip_reason_reaches_the_step_log(self): reason = "author dependabot[bot] is in [commit].ignore_authors" with patch("sys.stdout", new_callable=io.StringIO) as out: main.render_step_log([skip_scope("PR title", reason=reason)]) - self.assertIn(f" ⊘ PR title (skipped: {reason})", out.getvalue()) + self.assertIn("::group::⊘ Pull request: skipped\n", out.getvalue()) + self.assertIn( + f" ⊘ title skipped: {reason}\n", ANSI_CODES.sub("", out.getvalue()) + ) self.assertIn( "Skipped because author dependabot[bot] is in ignore_authors.", out.getvalue(), @@ -3142,7 +3688,7 @@ def test_a_passing_scope_has_no_skip_reason(self): def test_older_engine_without_skip_is_unaffected(self): """Back-compat: engines that only emit pass/fail render as before.""" results = [pass_scope("Branch", value="main")] - self.assertIn("✅ **All 1 check passed**", main.render_report(results)) + self.assertIn("Commit Check passed\n", main.render_report(results)) def warn_scope(label: str = "Branch", value: str = "jsmith/fix-x") -> main.ScopeResult: @@ -3183,9 +3729,9 @@ def test_a_real_failure_outranks_a_warning_in_the_same_scope(self): def test_a_warning_survives_reporting_on_a_scope_that_also_fails(self): """The scope's status names its worst outcome, but both findings - still belong on every surface: the count, both tables, and the - details block. Filtering any of those on ``scope.status == "warn"`` - drops this scope's warning the moment its sibling rule fails. + still belong on every surface: the count, both diffs, and the + listing. Filtering any of those on ``scope.status == "warn"`` drops + this scope's warning the moment its sibling rule fails. """ mixed = main.ScopeResult( label="Branch", @@ -3212,15 +3758,33 @@ def test_a_warning_survives_reporting_on_a_scope_that_also_fails(self): self.assertEqual(main._warn_count([mixed]), 1) body = main.render_report([mixed]) - self.assertIn("❌ **1 of 1 check failed**, 1 warning", body) - self.assertIn("| Scope | Checked value | Failed checks |", body) - self.assertIn("[CC202 merge-base](https://commit-check.com/rules/#cc202)", body) - self.assertIn("| Scope | Checked value | Warnings |", body) - self.assertIn("[CC201 branch](https://commit-check.com/rules/#cc201)", body) - self.assertIn(" ✖ Branch (1 failure)", body) - self.assertIn(" CC202 merge-base", body) - self.assertIn(" ⚠ Branch (1 warning)", body) - self.assertIn(" CC201 branch", body) + self.assertIn("Commit Check failed\n", body) + failed, warned = body.split("**Warnings**") + self.assertIn( + "```diff\n" + "@@ Branch @@\n" + "- jsmith/fix-x\n" + "# Current branch is not rebased onto main.\n" + "```\n" + "\n" + "Rules: [CC202](https://commit-check.com/rules/#cc202)\n", + failed, + ) + self.assertIn( + "```diff\n" + "@@ Branch @@\n" + "- jsmith/fix-x\n" + "# Use
\n" + f"{group_line('⚠', 'warned', 'Branch: 1 warning')}\n" "\n" - "| Scope | Checked value | Warnings |\n" - "|---|---|---|\n" - "| Branch | `jsmith/fix-x` | " - "[CC201 branch](https://commit-check.com/rules/#cc201) |\n" + "**Warnings**\n" + "\n" + "```diff\n" + "@@ Branch @@\n" + "- jsmith/fix-x\n" + "# Use
\n"
- "/\n"
@@ -3325,30 +3895,43 @@ def test_warning_only_golden_output(self):
f"{FOOTER}",
)
- def test_warning_and_skip_both_fold_into_the_title_in_order(self):
- results = [pass_scope("PR title"), warn_scope(), skip_scope("Author")]
+ def test_warning_and_skip_each_get_their_own_group_line(self):
+ results = [pass_scope("PR title"), warn_scope(), skip_scope("Author name")]
body = main.render_report(results)
- self.assertIn("✅ **1 of 3 checks passed**, 1 warning, 1 skipped", body)
+ self.assertIn("Commit Check passed with warnings\n", body)
+ self.assertIn(
+ f"{group_line('✔', 'passed', 'Pull request: passed')}
\n" + f"{group_line('⚠', 'warned', 'Branch: 1 warning')}
\n" + f"{group_line('⊘', 'skipped', 'Author: skipped')}\n", + body, + ) def test_a_failure_still_fails_and_names_the_warning_too(self): results = [fail_scope("Commit 1/1"), warn_scope()] body = main.render_report(results) - self.assertIn("❌ **1 of 2 checks failed**, 1 warning", body) - self.assertIn("| Scope | Checked value | Failed checks |", body) - self.assertIn("| Scope | Checked value | Warnings |", body) - self.assertLess( - body.index("Failed checks"), - body.index("| Scope | Checked value | Warnings |"), - ) + self.assertIn("Commit Check failed\n", body) + self.assertEqual(body.count("```diff\n"), 2) + # The failures come first, the warnings under their own heading, and + # only the failures get instructions. + self.assertLess(body.index("@@ Commit 1/1 @@"), body.index("**Warnings**")) + self.assertLess(body.index("**Warnings**"), body.index("@@ Branch @@")) + self.assertLess(body.index("@@ Branch @@"), body.index("How to fix")) + self.assertNotIn("Branch name.", body) def test_step_log_prints_a_warning_annotation_not_an_error(self): buf = io.StringIO() with patch("sys.stdout", buf): main.render_step_log([warn_scope()]) out = buf.getvalue() - self.assertIn("::warning title=CC201 branch::Branch: The branch should ", out) + self.assertIn( + '::warning title=CC201 branch::Branch "jsmith/fix-x": ' + "Use/%0AThe branch should ",
+ out,
+ )
self.assertNotIn("::error", out)
- self.assertIn(" ⚠ Branch (1 warning)", out)
+ self.assertIn("::group::⚠ Branch: 1 warning\n", out)
+ self.assertIn(" \x1b[33m⚠\x1b[0m branch jsmith/fix-x\n", out)
+ self.assertNotIn("How to fix", out)
def test_step_log_friendly_line_mentions_the_warning(self):
buf = io.StringIO()
@@ -3415,16 +3998,12 @@ class TestMarkdownEscaping(unittest.TestCase):
"""User text in the report must not break the Markdown that carries it.
Commit subjects quote code, branch names and author names may contain
- pipes; the table cell and the fenced details block have to survive both.
- The checks below come from a real ``commit-check --format json`` run on
- the subject ``fix: handle `None` | retry`` (all rules pass on it, so the
- CC001 outcome is flipped to make a table row).
+ pipes; every value lands in a fenced block, and the fence has to survive
+ both. The checks below come from a real ``commit-check --format json``
+ run on the subject ``fix: handle `None` | retry`` (all rules pass on it,
+ so the CC001 outcome is flipped to make it a finding).
"""
- #: Splits a table row on the pipes that are cell separators, not on the
- #: escaped ones inside a cell.
- CELL_SEPARATOR = re.compile(r"(? main.ScopeResult:
return main.ScopeResult(
label="Commit 1/1",
@@ -3442,71 +4021,71 @@ def _scope(self, value: str) -> main.ScopeResult:
],
)
- def test_markdown_code_wraps_backticks_and_escapes_pipes(self):
- cases = {
- "plain": "`plain`",
- "a|b": "`a\\|b`",
- "fix: handle `None` | retry": "``fix: handle `None` \\| retry``",
- "`x`": "`` `x` ``",
- "``x``": "``` ``x`` ```",
- "x`": "`` x` ``",
- }
- for value, expected in cases.items():
- with self.subTest(value=value):
- self.assertEqual(main._markdown_code(value), expected)
-
- def test_table_row_with_backticks_and_a_pipe_keeps_three_cells(self):
- table = main._markdown_table([self._scope("fix: handle `None` | retry")])
- row = table.splitlines()[2]
- # Strip the outer pipes before counting: "| a | b | c |" -> 3 cells.
- cells = self.CELL_SEPARATOR.split(row.strip().strip("|"))
- self.assertEqual(len(cells), 3, row)
- self.assertEqual(cells[1].strip(), "``fix: handle `None` \\| retry``")
- self.assertEqual(
- cells[2].strip(), "[CC001 message](https://commit-check.com/rules/#cc001)"
- )
-
- def test_truncated_value_with_an_unbalanced_backtick_still_closes(self):
- """The 60-char cap can cut inside a backtick run; the fence is chosen
- after truncation, so the span still closes at the right place."""
+ def test_diff_fence_outgrows_a_triple_backtick_in_the_subject(self):
+ diff = main._diff_block([self._scope("docs: show ``` usage")])
+ self.assertEqual(diff[0], "````diff")
+ self.assertEqual(diff[2], "- docs: show ``` usage")
+ self.assertEqual(diff[4], "````")
+
+ def test_prompt_fence_outgrows_a_triple_backtick_in_the_subject(self):
+ how = main._how_to_fix([self._scope("docs: show ``` usage")])
+ self.assertIn("````text", how)
+ self.assertEqual(how.count("````"), 1)
+ # The commands quote nothing of the user's, so their fence stays.
+ self.assertIn("```bash", how)
+
+ def test_a_subject_with_a_pipe_or_backticks_is_shown_as_it_is(self):
+ """Every place a value lands is a code block, so nothing is escaped."""
+ subject = "fix: handle `None` | retry"
+ body = main.render_report([self._scope(subject)])
+ self.assertIn(f"- {subject}\n", body)
+ self.assertIn(f" ✖ 5584f46 {subject}\n", body)
+
+ def test_capped_value_with_an_unbalanced_backtick_stays_in_its_block(self):
+ """The 60-char cap can cut inside a backtick run."""
value = "fix: " + "`x`, " * 20
- truncated = main._scope_value(self._scope(value))
- self.assertTrue(truncated.endswith("`x..."), truncated) # cut mid-span
- cell = main._markdown_code(truncated)
- self.assertEqual(cell, f"``{truncated}``")
+ scope = pass_scope("Commit 1/1", value=value)
+ capped = main._scope_value(scope)
+ self.assertTrue(capped.endswith("`x..."), capped) # cut mid-span
+ listing = main._all_checks([scope])
+ self.assertEqual(listing[3], "```text")
+ self.assertEqual(listing[-3], "```")
def test_details_fence_outgrows_a_triple_backtick_in_the_value(self):
- details = main._markdown_details([self._scope("docs: show ``` usage")])
+ details = "\n".join(main._all_checks([self._scope("docs: show ``` usage")]))
lines = details.splitlines()
self.assertEqual(lines[3], "````text")
self.assertEqual(lines[-3], "````")
self.assertIn("value: docs: show ``` usage", details)
def test_details_fence_stays_three_backticks_for_ordinary_values(self):
- details = main._markdown_details([pass_scope(value="feat: add `login`")])
- lines = details.splitlines()
+ lines = main._all_checks([pass_scope(value="feat: add `login`")])
self.assertEqual(lines[3], "```text")
self.assertEqual(lines[-3], "```")
class TestSkipRenderingEdgeCases(unittest.TestCase):
- def test_failure_table_omits_skipped_scopes(self):
- """A skipped scope has no failed checks, so it must not get a row.
-
- It previously contributed a row with an empty value and an empty
- rule list — a blank accusation under "Failed checks".
- """
- table = main._markdown_table([fail_scope("Commit 1/1"), skip_scope("PR title")])
- self.assertIn("| Commit 1/1 |", table)
- self.assertNotIn("PR title", table)
- # header + separator + exactly one data row
- self.assertEqual(len(table.splitlines()), 3)
+ def test_the_diff_omits_skipped_scopes(self):
+ """A skipped scope has no failed checks, so it must not get a hunk:
+ a blank accusation among what failed otherwise."""
+ diff = main._diff_block([fail_scope("Commit 1/1"), skip_scope("PR title")])
+ self.assertEqual(
+ diff[:5],
+ [
+ "```diff",
+ "@@ Commit 1/1 @@",
+ "- bad message",
+ "# Use (): ",
+ "```",
+ ],
+ )
def test_empty_result_set_is_not_reported_as_all_skipped(self):
"""`skipped == total` is trivially true for no scopes at all."""
body = main.render_report([])
self.assertNotIn("skipped", body)
- self.assertIn("✅ **All 0 checks passed**", body)
+ self.assertIn("Commit Check passed\n\n
Show all 2 checks
\n" + "All 2 checks
\n" "\n" "```text\n" "Pull request\n" - " ✔ PR title (feat: add login page)\n" + " ✔ title feat: add login page\n" "Branch\n" - " ⚠ Branch (1 warning)\n" - " CC201 branch\n" + " ⚠ branch jsmith/fix-x\n" + " CC201 branch (warning)\n" " value: jsmith/fix-x\n" " The branch should follow Conventional Branch.\n" " Suggest: Use\n" + f"{group_line('⚠', 'warned', 'Branch: 1 warning')}
\n" + f"{group_line('⊘', 'skipped', 'Author: skipped')}\n", + body, + ) def test_a_failure_still_fails_and_names_the_warning_too(self): results = [fail_scope("Commit 1/1"), warn_scope()] body = main.render_report(results) - self.assertIn("❌ **1 of 2 checks failed**, 1 warning", body) - self.assertIn("| Scope | Checked value | Failed checks |", body) - self.assertIn("| Scope | Checked value | Warnings |", body) - self.assertLess( - body.index("Failed checks"), - body.index("| Scope | Checked value | Warnings |"), - ) + self.assertIn("Commit Check failed\n", body) + self.assertEqual(body.count("```diff\n"), 2) + # The failures come first, the warnings under their own heading, and + # only the failures get instructions. + self.assertLess(body.index("@@ Commit 1/1 @@"), body.index("**Warnings**")) + self.assertLess(body.index("**Warnings**"), body.index("@@ Branch @@")) + self.assertLess(body.index("@@ Branch @@"), body.index("How to fix")) + self.assertNotIn("Branch name.", body) def test_step_log_prints_a_warning_annotation_not_an_error(self): buf = io.StringIO() with patch("sys.stdout", buf): main.render_step_log([warn_scope()]) out = buf.getvalue() - self.assertIn("::warning title=CC201 branch::Branch: The branch should ", out) + self.assertIn( + '::warning title=CC201 branch::Branch "jsmith/fix-x": ' + "Use
\n", body)
+ self.assertIn(""
From 7cc4bd05226fbf6dc301e057371a50bfd15d796f Mon Sep 17 00:00:00 2001
From: Xianpeng Shen <3353385+shenxianpeng@users.noreply.github.com>
Date: Wed, 11 Nov 2026 11:23:43 +0000
Subject: [PATCH 2/3] docs: add before and after screenshots for the pull
request
---
assets/pr/failure.png | Bin 0 -> 389241 bytes
assets/pr/many.png | Bin 0 -> 420651 bytes
assets/pr/opened.png | Bin 0 -> 819699 bytes
assets/pr/pass.png | Bin 0 -> 112577 bytes
assets/pr/step-log.png | Bin 0 -> 517052 bytes
5 files changed, 0 insertions(+), 0 deletions(-)
create mode 100644 assets/pr/failure.png
create mode 100644 assets/pr/many.png
create mode 100644 assets/pr/opened.png
create mode 100644 assets/pr/pass.png
create mode 100644 assets/pr/step-log.png
diff --git a/assets/pr/failure.png b/assets/pr/failure.png
new file mode 100644
index 0000000000000000000000000000000000000000..5ad74a2da6548873fb95b2546c48cf231a716bf1
GIT binary patch
literal 389241
zcmeFZbySq!*Ej4}{i1?^w6xM8AT6zQO4m?Icjr)|QbS1$NOzYE9RmhAbPQbrLl52g
zT#Ekgdp&=?>sjx5;~v)<KfG{-q8zj#R*4
zZf&wxsN;2NRA>Z|-Pj&#nkCL|tRe93&6<~tUaE99OI(XSc!+Q3Eu<&o-T04h!^Y~W
z?9Jz%iO>x3VtW5l+=n<_cI6S(n#JZ3=@U63Ow08zYJe3gxQRF@GkdwmDL-0Hf-c5qgW{H~YEC)^VM|*NXCc-V~18-y~48
z*m@qdmO}%rtvX=NfLBk~*47~IXfjP1TynZUCo;zMYKT`VJ3QOhQTB6|@7`uxvqc#P)cyxlqb%~8tmlP6xy1RfRw*&FU
z8Y+T74KTZe6;sKZTjHo*eaoC}zJj=K5G&zcdvh`Hy{?K%7ftub)C7T(vfi4Ry@C?`
z{@bst%enP#Zv;qI`p}?lfs-iX;r?s6$RAu>T({r9zw<$y`rh5VoSex?ood_B@tzS$
zH$`LPn1tGL%aCY9UqQiVJ*s0ObP;qW;vu#sbxcyy*^4Kua&H_hb?TtGQo0uTd#Ds8
zCB+ftC%}B8qtdg`ORMv59Fh|gnIkU-rnFO()2UJ;!qWttEp|$n)O23P77vo-VP=0=XVdJu_ogs2IuuRnX2-O+D5TcI7z
z&RjZC2bcQ+&JM!)@|<7&K={GKG(c50EBNi_h{PaADb&r5?284K`FVNvQ@)PZ(I!RR
zM$A=~Lyjk(nziDIi&Firl;iGg&oz4Qw;x*cp3@4`s0|L6s_U6$B4^GGSx4gsC2QTbHW;?{@YtjzrUTKBH$7XIyN|0Pdz^So6n;7nnb-l!R7eA&TX9Uv~4L48cJ
z?RHGdXt{o^u9g{nh|c;&u8q;LCm3&*|J^BR?_57j(~R
zgY$E9FAt{tHMNRfr@>Dy%fn1HwJ>p;np!Tsnw=4}H{(EJYRmlr^~f@4l-8uAOg(e6
zVIhgb5B-!CQ~0%wk~dA)k^@eb_gT8R~{*
zL1jZ~wgp_*)|J}KQIN9g$}w^6NsrYz0t8tScyzfaGai=
z{@P_PJnMi(m}W2GUj@IU)5K;!oUmHzvlU5WHJIhNj^4!71OuHg+imYvbnR04A!59{
zR9wJgY_FlAAy*h_|&xIw%NPtkvLh
z9AMee+CEd~tDQ*8e6Tsm$;PI{b3}C}ql#r_Za!I|#M)*UjRkdadM&Fc?7Uf(l@HjR
z{dR20AK$+#dVOl^9s{4Bu}(6yOIKTKxO}O_#ty4PIdKRGNVyuepelUO?M}3FYGNw~
z2M5?w?qNXz&COv7q!M=|OlWEU*xi4N08<#Mi1=j*Bb86-L~nIUmU`D>^GaJWKcQW8o|2z~>4%1D!ej%%rmOu6F(Mn?fA0oT0Ah_|M`@e;n1pRe2EHqUD
z9&1U?l};b>J?3mwYV)!Nxz}ce7i7``pn)wqU&WcCcbp%Mq
z3B61y)tIFhXcgN5(giW`{rFDnms0tHiB6+RIi-1^QI9d73nwONX(`TdM(A)G$=2~8
z;6@ugo#+on1A?zbs%@_E+WH3Syf~@`5Lr`+SC;hdqh?#qas)L-68W&O-J#rhVMpd{Vf-|9f
zMqW-1iWvvPP|())Sor)C!NSVy{i-Bd;+&OVIlU!r5wF3`mb-47TtoLpHwn}5L6*?$
zkFBkwVtuJ1{&KRGb=KE*QRk4L$jH~-<`Je*u;={6>cvS}=+jO{T3U&r$PN)@sUMvK
zU!>Ge-x{QJYq)xYUJ()E7uCJWRD{ZJx!3T)`jS1in5RmU__h2simzKfj#TmV2aEn^
z_G>QJy~FD47W8Iwo8{sHBvL{G^r!{shT1p004P_evja*Z8{+jY`xnu^DEzRd&2P3AALV9zSmY^Ga{z8`$ZH6c{G@t{3q1_jCHr1%&
z3)I@(4Mo(1RyakP*xHrEp}&O49{z*y#wwCO^|xj)o0C>S#J@!f3=aja73
zkz&1$N|G`r1{|=+9`-{VC-z0ANdb0#Xfhpingv&|;hJ6@ar&MMeFU&$!oFK-wXf}R-)9sjN4c`{CKUWiR)JR_
z+KiUvadSsRgppvY6U3sh{9`cRqpltFnNZCo8kQ&i#X1u@dUBJh
zscR>T;Z_LpCpQsa#^ccT8FHeQTF
zkBsD39<#)`Jix>uiU7{Um!11|TMrAOtD(8|iB|eP?=8h1JOped4>xagM7{T%!bI6B
z=+)5cz&&-U-kX|w*i$AYjjC=kQH|z5xnpu2zv=~iV
znISr#S55i$him=QT4z%rKcO!bfI24^yO
zX;ff+qO(K3dnr_IWJq&^t*1kySWUHplz_a}`f?O2q1@^M3F+I}_UMr&J}^wHN$lk9zF6lgTC)DqQ8A{%~Y
zjNm{SPtgOdK8&8^w6wC=Y>79HS|fiQfo
z&rl^dCV3%1)Gtb1O^wJ-0c$_aK756krjH4#xBvs9m>qrEmq1lsbhV_#0Z0}@PM9;fi;=Cn9!
znpeq!ej`BC8aAS68x$lJLOt<0(oPTl${OSM+EnQJ3>u1T`K?s=qoNrY+TEDjPKKLWn+nr9mqd5$59b6-(ed2F~FGod|#jD@BIE1@#J6yJo()2o}x-rW2Q
zRTT}$}Wwa3Ot4G0BM6AiVs=?VE3K|V48b7_g1GeH_e;<)>IyW7oVg+8W58D*cjE%zo>
zxQ$)KT5vEhFcfYTsaC6<^Od&RMBfaqass5)TOytX)3&p9uVxpM`K>aZRSA0Rt`}rp
zy1BX0vyBw$qAPLRISi_w{>nvY)t*teFhZwULux{r@|yV9<2x>Ug^qv|r|Fvk%?*4}
zY@a%{Trom?^L)(Y$aTU7Z3!R{3(0ET;Ns+~si>){s2So~N9Y0O%6W!a0n24<&n)--
zr|9UPrvr@iyvUbEMkz(9rAmHd9e+>r`oznBgMcYk9m^Z4il(BP!Qt`92A71wVXE^Z*Hv4LfM&j^C84Q
zt}oA(OF{Y#T}<((kky@naeQado6jD(y1H(buu<5R9vm8?po3k1?vc-kiC2P#h!=l8
zk!*RV(nj&hn_NNNg!ayz%OP&<8}-P@!F#1-Z4DAvbJj5pA0I&?o!s2NJm-!!lrmMn
zc|5&*Xm1{*Wq^f;i{Uws8joa<=IG#Sn(sFX#3aC#+wtvfXcu$V+JdExgWXd&=x__n;A7ZahVJr26TRqd!~
z)10CGue*QFt!_dPVi>=J?6%;DxP#O61Ugs0c%MDsua)rtp03X!%q;n#ew`sPGjvxZ_!jCyNJdl1H-
zi5ud#2L&z6wQd4W&FjnyhZ*7A3}=YhmZn5B1<@i+Ok6OcoG})XJQx$s`ym$BZ{v@j$LiO%;z*7YqV}wYb@yaxzQCC`Lp8njaqV
z@aD!_aSVSpxakRI>AS2?DKW&vq@^Jh2d-{P;n=;35!i>=2M4YPZT^CSYCj*njqijF
z4Rgn;s}sb;;N!=(3DDi8PfmP1HK9XBy7MP%m`BRo%;sglyc#i+t5c}ltITK`?I9$x
zx)S!Kh9?#R3F&%xa42PNW^4M!%$^0DWxVh`+hma-R#L9~<24N@Atx`k)GpVm-g*BR$ylT_IP-1R5UiP#pO5RcVW-T&G3f5>KG|ZC
z5E>eKFa&{kC7DiubjNk|Wa&RNry1(XObv8Cb^>Qc55`2A+PzBCgq)m-g@#4(bMv|t
ztK1;Kak4klx&KKC>U^^)zh|`BGr^5GsxV=!dRXaq>9$qBze0qDvc}g~&g!#J6e;tZ
z=j)1iAAK?;ib@yOqSF;uc{5O|=aIx`_Y9u05E~=)hK?+mZcSineGRmuw8NcQfHWFOtE>YcXQ8cS7!
z>)Kv!cLFvxc6S2nSe6-<0#;aD+=$IsIqT8-#u~5nh{~Yfh2(veDoTn?)`rG=ckiBv
z`7La0WDA27Z8rnM5+r{N4t|^IW$Um2qDGSbRn*~g28KNIUNXC>s%Y#-mmb?Q+Eoz|
z;p@oa;s*W~OyeE>%7ZmEHDy}WkT|Se@xD~CM$a3lP&gloy|89S0Kj`)%~0
zKYbclU9~hYfD5`=R|6_{m&Z$(j)&(Uv$i&>R#{T=qi6aCvUDm(&II|-@s`*l87%B5
zl_TK?_f?uApe&kIwUq?eZF%CCXNTbrvCaDNsnh2HqyW80PDRBaN~S%AX>V^&gBydb
z?(2X?qggb2K!+l9FC;09gbM=#{L)3?Xj{8n{gdwIn|l_cz^dF>Bbu5K7_%K;KYsD%V)bnV`DnLy&I>uGBES+3R_vZ)R!SPLEFPc8
zy?g_px+j|~4Nwo)FJtCD&hqlQB>52gvSoo@%m_%N7ye^9GTtIBVxz_Jx)UJ}>W_he
z+65{#?h)1q4tI0&)px*y-CZ6CJH89fYy)bYUZhf_YZN!N`A5R#~B6&`)D2)f-n*
z;v#^>X!P_n&ET;2;Bf!gpTg4IoTa_7QS;_Bw*}j>7;aZj4>k1>@huObA6J~V3|
z<93*7DBY~^a6EBe4xg;Hj-=oP4Vsvk_#Pd|NqHY{iC~eiX;t0BS-tMXNZ~%_
zN1DvmIjQfC*9JYjTJK(4S7w1#DvPF5#rf)%4U|~Lax|Yb$)gYms^}Uf<(D~Z-p897
zBHzV2B7Ki9qag0VIA{{K6D34fZ*r|@`W1@S!NDvlxp+$3$afYdg|$}id46&s?A_ND
z%P3`-G0Cx)cia)RJ<1k*YS-*1#{JUdFBZ@|=gYs*Gf@fi!~5xb@KXboc){oYGn9Z)
zsEb2zL8iaqEi?|zrlB)`G-4vAO9dS-n2#AAH2(M+_>j9^Xj@i~lNoi=0Dc_gL1No&-A6k;qfa;Rn{?keC5(fD+`PLW%3yij~x?v
ziRXGBdg@?(cC8<}VedVyzJ*_2V6?q8BUr6&EBfGmQln#`^ENPtwkXRG9lN2r=>FqP
zz_0F2V(YO3_1wvc5~oWKnI`(*W*kB>q0#(FDOw2?VJjbc6TCrmDwM2PpEG%WZNBcc%VZ}FI=%uX_WAlfV%
zoA2XzeniQ9eR0wHPY0w^j`;BdTx!v?okc7Et9p%5|MSR>PqjgXjJofGJ7xyABZWM7
zJ{^2-ZM#_IG~u_K&GIu|@V%PPw(5>A??hLuC6X>w`KNL#Cz{Gl431BG?YEpBc_X6(
z0s&*Td@^1TR=DelH2%msF
zU-W7|+61w?(1AdxZ8o?sj`r^pceF7sok)_f8_d95(qHfc$t^Z^rTJaR_SYEb27vnZ
z=v67w=>l0SU__YG1fK~n)XpUUb*mz38Pm5e#O!$6pJl(LP*4)3p5t7)bT7`rkmQmv
zx%v5e06xQ|e!kMW2EP}Dz@bSy>!@#Gh0y_(moZFK>!9R35fv5S%g^i@j}!!2v`Jn3
zjaXuyIXUh=irth;T)>`aa7pJf=i=kjYGQBs8qD
zJ5V7(c*~PXaD_vE1ZEc2;I_fUM=j>XwFTc@FxIPJX8!{C*|61D^*jwby&hsbc9R-S
zCYr?>ujO9*EiF<~5SEOime-e4L^xD7dlyS25!`Qwa+3KSn7#G&^$~GS2YUyQ2ye2~
z5L3jr*-Yhh!*yI-983Yv9mhzS*#^7w&gdURVYOH^J8Egko*NjrWK6K24Ssw523Os|
zdcVt{NXp4jjt_BRbstNFJHNL|B5M`IqRhhEmIc8s50i+O#Dnf2L#1Z)adgOtgNn
zuLP9e9?A2pBMSw59L4aOWhZRHsZWnbwl0Gue*CzeZSuWX{kgulFR+t6kTkk}n!EdR2uZ$Rcd;JJY+t6^Ql#5oXX>UR+c_CA
zCHa`H&$_3emT-IjP>-tn>P(1P;B>V@9WZPvsZ=b+T(8mes~!QL6Uf%S24jdP<7zH9G=a4XT4lwTVB
zjzp3Vn!Xqf7`I>lewn+MU5D%CmTtQXsEsiPCVr_Qr>)Jxma#HXuZ6inbp}@2#}Tuu
zgLWuLF0D+Y#?eTTuKGbpI^K`q!l>heJ$!-ANVFb@!HeVb@24~{)h$7fqb3bU$vEye
zFR%A4luPw0*AEI8wbuAKk6XK9PX&`|qm&&TzZlo=Nk@=TiTLc)*1L!%9u3y?
zk`vkFvr?$*8vCLUnhRWdgEbWuvsFL0;ok1{b2VniW#}>@eWFWMRaJ6+JNyV-vClNK
z6{Bo=EY$3xcx_YTO}drBuQvyNW{;JNTvAA_?+!bpDm^r`!N`=)Y0=w8*Q~sf9JdnT
zJJ@*&7~?Fo({aF=lUj_r=;xC%nl1!bIR){MbU?w0LP#k2xFb>{NC@ByZ6dQPEnC|d
zPfkR&s9t^4KNhK+ovjT%P&b!TJ6d0*Rwga~{15YczQcUt)5#N$Am-R-nd*(BY{|Sa
zuzUdwjFm$v@Ro%?#Us~cajhd3(d@462sJw0_tj9;z3mpgVz6ek@NazVU6wc>J?h>s
z%RBqyKc?3ce#Q#A^xpgoGiWdD@tEt;=(|hCe>B5w(a&P>fvAM9RL@^zQ9-KF_uL&L
zD9EHELfbYdFgW<*M?pbB*?%*$8qa%3jaV%7frx9Q(xM+c4bLAJc3RNrz@e-XOKMD>
zy7Fz5q9T0s=#8zZZG0+}sR~?*yt;Fb2px=@UWSpLl+@s{EsA;^jXPSXt~XewR=-y}
z=Obb~BDCRyXR-v>v9vr(k*8T`{&2oM)l5`6$hA82aLw6{U
z{rYXg|>H#u(s>qen!HN==Jg{un;-a7*m&CIkW6O&P8`t_j@0trr@%VVX|)kO=Y
zt)a*wElpu9H|^eaO`?cTUmT^}87RaO-8KS%SfiZRfkvnKSlhrZ90UGseV^Werb~w
zU+zmjC_jw(U)XQ#i|MD~B~hvU?3u%>?PlLIrsW^ZhwDgn7K+{^eqZxSD|KWsVqid}
z@Lfp4lQ3=Y@s)$`p5C?pDUi_cS|fYWGH%Rn%&cSsG
zBtBg_uIT7N5)Px-w1j5E(Q@fL`S^tFb3qNGAhter_pC!aU&GwB9l`SMIz;83)xjC*WrTn>Lw;qv;y?H
zgO3u&K5S~MDwyr17p((yMlw;qE0}?R`6#;?f{6?c=XJwtz%T=Qu?aDDDzeqQ_(9x<1`T?jZ~^<+!{Q<|&;kdqf!vHL>O%Q|jLB
zd#&3D5Gy3jSrl#^y6%(QUzSe3&xH$AfFM$PxR);C7fYO6EP~RbCYGOn%;~_d`_(lr
z7l`I7J4`K1U)T35)YR679edwEk&wF+adG~DQ28d}1Yw23(lIw~AI}qzsaiyYB2w#q
zZBKKc&s!BEqWVudh48yb3Z>SUr`Ua3W%|g=h*`ZFTM>^dpIN#1PcbPm9f^d5P9o(E
zR%TZ8ldIJ%ZfKw`gpiqXWP@eI&gFJxYMH8g(pZ&O^r%Mg^p`&
z!sTA|%EGhsc{6+SqRw}K*|*qJ
zp;J@+#PDH{i7}6ICM&TO$Um;4;@@lhw8aT^b;510GXGg9_YD9i_0ASzD^HU=bbCU>
z{*UfRdi`E!@%#5ngW+IhPBrCJp!Cfn%0eToHcCKzG^hB=#f4icl6EMuMT@-~{e%dS
zpv{9rExOm8QjfSILl(o2CT-f=rQ_q{F)%Q=TTWjey>!9iHWT9Fl4qe<&@8YI3J%uO
z6C>aY48A0J!~7$9Wwg+wLyJw%upW#a%I&u3OFP`DD{eUc8X$q{hF)v&Jnj681-w~b
znx-nt&reQbfMqJ)DfNc~@Nz`>L*#ksGd8IhB5G<5GhfmnqPa&33NDAy?lroxPZl+n
zwU)I(prD1NeJb;~)DLuY4DyqMFGQdITw^B~#!ve4`LS4P_C%5HqPY_JGUrg&H$#?x
z`t5Uc3PtJZYooggnSWOvG75etxg--BQ~=2GSndY*%
z7i@`mY`xbgInXday8G|FuqtskzvTS*UFDKCXB(9#
zs5pJb8HYvH*zJ5*2i_h6O5KNWmz9-OR16P+$BB@Vl5SiXZZ9lsE~sfOnZXq0z2hfZ
zzY!&SY|oY-Y&M)xd0clVBcQB|@aO94!=vTZ_+r$4&*_<-uO8I9kn`KCHCK@H*{1L~
zy>IsGYY_!-VYx15fZy6KjS=GxSihK9^^yIO&*LT4y%7&9&khd{CDT49C6N&ksY~0T
z(4CvP^=92QD+2conR*ekXUHkUimg!p9aRx*nx8J`~QmIu~Pgr)do;&94}e(f6x~i%h+$!R50zcgogW`ddubd
z>#$9fd{tbsQZk>6j0`J30RaKO!%W_^?f93>IO-ys*f*>F^~d@jT*Tv=AJ34B>&sZk7)WBCR|wglIr
z2S?qz=J$4YA51D|KYMRczPij}*0M4yD&V7OKL=0$9M{6Z_)F;XQlFIMPG13z9(w#7
z#(W6BfvHvE3@^ot?G1#43KT2AE2*3``7c~ozuecw!1iMmrlB#CsjRJ4uh|A-sjKgS
z;o;$59PKPr3SC^>c(v*oFB#K)n;i#@@fu2XPRG8yrOi?7^-A=7U^18NcpfR^>FdX#4H}P_rh;*)bt{~qXP(jS*rdWb_4OxFVq+x+j5JS~
zZ5Za@M#~p@)5vB!J+rCTC_7=dp063D9lN&2D
zpgx|xixNBnyqm*ZbLv6U0s%pu|1K(P?s~k!X6$`nSea2{=Cs@Otgo3~tKLyPN2{H+
zaf)w2PI!+`YwC5%3$_gb-z{S`XINpV98!J@k_P07^19pOQWKp3yWfSRJVg3ZkQbT?
zS<45dy_c24huy&;Ai#dBRoRZM2s4^5^9>%Ugc_XVq}MxoeWPZDw&?aI^YoZp%-)fS
zt<=Wb9o)H@c#
zjVJ=3(RR(cRMcPZwA>h^`_cc|M%n*u8(Hy)l+t)q>zT_?lCL=re?&=^$n(;yYgyyx}(yQ%=^plMgFV2dJI~WJrTSL*V
z&cfn{)|w8pO}uaB=g+W{4>rV9+(4$o3`$A7Rx~J?h;`mLJG=sHNZ`Zl1P}b{XK0`RT)TTBKo19J2ZcfYnI(9YoGv2(GBL}6m!D2oKvJ=;b;T)}cEF$IdM63`-wJHe-@!%#A
z+t$v(?Qkkqve-PI3<|!@(prrbb;RoW>SGIzDk@5H4l6Xk?4+4cwVyqzt2OH|9JHK)
z748vL``A&0OJ-%Ev#(bL|tr^_-@QXG1R#{i8^GO92!^z8Qe
z=|bFPo^pc5t$wJ$K0tAgtc!c+v2w}&C*%dXeFtx7mcEn9_
zyg$HJ9)SyEh7~r+#Qa4pc83z=1Fn4M9%5k=JXN)}F0%HowKo8;gl4OKVl*yFr(Ko8
zZlf}+VGi!C4gQB(G1d8W9r>m@dDH_vSq}u|qnB{QibQ6V@U_i__u|!wOBehc*YuK|
zoxRfkOjSmm?<9sYLEpM$gUhU5aDK$liXxs1*{H
zdtmGbnD8ZLYW+G-xeBk{A^M4TN9ZYRaz38z592zY|7FORl+AmO*z^QZq>V^RW{>!bb(5)6}Co}#HXYHiOUCo
z^#^oCIAsT|uU#JNW3M9{oHqJGhga;qrYmbJPd$gci@Y@fZc2zc5w;ChDOY498Z@`>
z<9fMl*B14{%fKMDvortret}``-avrEvdC;}r;emE1a6j^8ic*Y2=$~5#<}pFV8QD`
z3pg$Tkg$7AjbMXd!1Z;tTdB%xAD*EwtGT>fpaQVE`q`_6gCrR`ZIJITymp%AJD%b9
zPr$ckzbJ><`t#EsT{D9e$^7!|#gVEi96Q6X&@i|wD(D7`WieHsq)i3_yAUA2`^|2E
z3>LR!LBD%1wo?EAQsZ_rsi3PYP*0bMe0(*AFmv(0lv3uH%QYX)^W_1I1WJ-!L4K8xKuL7GWMRD)ie_m%quD?X_sFqxcOy9XTELWKQGa(
zhJ~w>GN(%}ReA2d=Lfs0M--QomRk5-YL**{Kt*7@>BX9*FuU%&W5#gp9u0M>BCp}R
zESJL~=)$C)Ta)K8aZmbG-l0FMGdU?KJp+Th7YqgiHCcWGiagpyI;z!1K6hst-Dcn}
zQ!k*s|R86!gyKv^IsH8(ZI2$q(g^s@Py
zTwZHSOoivQaNref8cKrPI@C({edSWOLn`B`k?Vz1GOCQpI3g=33hyR@=)=kdOO)`e8}b|oMeKOUzN02=U%i}cPoC)>~BEX!#>p!}p3QXHmHQfAb`X}bDc
zO79~{Fz(jy-mlXL&8js%J{@`!SwWu
z+C>_Jvwg3@oHR7M{+rZP#81Ub8xGeIG|=gs+}xQaF9m(>FpXk)lzc-HLr>_f?lBG<
zaLVIn3*>S(0J8(2dK048kwp-GC6>DYMgWupRHIcFlpIS{zEy;0A4MrCC>&!wRO;^p
zhy=B@6(d)wS5bHHWAK3(DQui3D*8(Kwuy0N-kE^Fi}8txCBA0@0w}qcpH3u!=BwUa
zN}Pz0NJzxk@Nnt;W=48?)XQ&E9UM=Y<(4>rzJO7!cu*t>w#1ze2%?!iq2g+~NB-P!
zs?(gDyrjpZQX*+u{iAtu95N=vi1u+ghEoZ&w|u%0VRO*7o>M}22~#wH>n%px64^8gHPFmDI)
zS^zCQ1IiuXh6c5tvI_mOa!N{aN(%C?mi2B@$oqq7t@MqLVOxt`e(v~PjNsd6Q7tEou=w}7zx0)4d8~e20<|RY&oxe
znV}sF4&;^i!^op2y-}W993jBq1#25?pu}Vd^@Cp{0ALbO#QXgpVg6;@-`n*2H{k96
zDP7>L0=>(Bv4H=(=5(f3v3G!dL$bK_ZsFyPPTjhNuE@ISh5aieOO+m05E8HlH=>(I
z{W7=*MNfVk;aj&n>j@qB+93bKCjKig;OGAl^6>wQ3E2V7H+>(sGGue|@~*u1dIEmP
za2UaNUp#sK?3q5xi=!iVlzs|1f5}*>OUF@pE!Xqse>73W@ju^GfoTBr+P9YqSzHRv
zO^qqDP`rKnb7AGIcdA?a61@fbS==wWIhUVb#;!1V=eO_)A3Fcx-DfyTNp&2MpY@H)
zQ4<+Lq2Zoc9_`9R%47uOf18t@sQhKg6J~uLA3qW~qdGnQsIk-D_mj^-R0Ja!L#YnVH?8bq$xh
zVNQPy%N+Pa$z>vBW@ZK`6eYc#^6eNOD4#V>eBMw9G@i|(2iRqQ3BkAiGV)($zS!Fi
z?NGj5MY?8hYgtgd@utARB!iLYy^LtHOk`@@%^Y@a|Ci|6e)>n+2&ieVTw@o0p?Kk{
z@n;TLOgU|x0pfT>L~a+Yg#!a$0A3uw!>HuV$4uYY4eQyVkAZ&2Pbz0X)RQXgHaWbJ
z*_f<1#aB*FMkf0*Q%+G)*!?1B>_;#}$;D+K#A`EBZq(m7pu(c4U2fO{zHEr7c1z|f
zMWZ)=%Z7=E5F|oGN$I;kir(1?DOg=SIk^k8RtgC5Z#--}k}Z{MCm
zdV%2j+^)~o_cCUW@@RjbBvez<)D!^c0Y_C@TB@GN1)v{9r13WjCZGFt{!L-){fC&*
zn1o)SF2%|<7W
zkqj9H0iAOF&7Ui*_O;sbQcKa_Ph9{6oBfxqd-;MVsQZGcy$Km#
z$z1*lx$oSjiM>5T=4XK3ub6gL0d%B}H*UY{3}R^F*J9BM2zWK+u_b!e50ubXgjq+1
zzKi<=V&HDhRV!GqMKWT0dAHW@lW2O^*y@kvIMEim$|@)n>%${U_Sf40o(JEJ3UDOv
z?A!-dhm94BdE0AJ!F1U}jH$TfE|65_`JWydvBt}`L4-@+KtBqjx8Ucrv!n$0_(_Yi
zKyNaHXeRf2h2NsN85H`63UBvn3HLRn5X<@Een*TMt9Gd-nv^GFyMy{QR6a-K{75-z
zFWOqUIG*jwuu|=pE~wm&TNf{G;kN}UO5!R=k-%n_gscEY}y&x73Q8;J{Zfk9)5_6Kjh~qHYYE0ywtg&MI?LFQNJq0T0
zES#82rk%rv?D@*+(8Cl#7e0FW>N@);vS~;1;0wBW^v~^&GD{%s$r397@gyj8OcC)g
z93>znbz5BS1rYTDcLsX;;2=@;d-0z>{q6;Sc?BI3XZ64CSY%M#tMgVq4{mL1yT0h2
z2CA(+mxmTxdxi%9-4Q87Nl7sxMD2CHIa6!IXSOTD7dM>b4CR>BUePLh%Y`c1
zF4u|`9EFn!==i@&o+3$f$xO)
zNOLg5&%J4D*PkXtO@XggZa(b#dM@mm5037(wvNo~mOKquu;c^(y2#J{jYB+3{CWA%18}_qJ
zsQEW@MP^}g+ffO>WSB7!S0-LOFfh|_qHc1%XKUB9I&bWe1E~x0wp846cnn`-Q($14Z9<9fn+TO=_Ou)CgcW3
z#|9HM2dxw=6&p031Bb;L;7yrGN8yX9?E-7B4*VE|08chufuc2anO)b%4KIEdx>e}p
zwKI_;QKHzIjXMNE52;?De-r`+r6`pxiJwx?^izKRSd`eM^Jc5ugoHbKb}CoBV{4{C
zmWj7D!2)q_c;vjdN0CuE<+**2rLKQBac}->zw47w^||Yl*s&inFt}LHneIm%nvg#}
z$62j?CvmeZHRAMmD{>evp&>LAj(t_6_1Z{AW&%i;EXwl9Q@ba%Po>g=NM5BKA;QAK
zVh;g^oZK(5@2n3&zSmjgKF>_udQdWjCNJ)hzE8o6tKV3&efgcl}q}AS4;a&;?I1$c5lc_>f
zQ9eH3zkgR(R|A-utLy8a!mP%eu6m$&bH;u<8c0T2_<>GS4ufWjM~?{!2o~q(=a)g5
z0GL-#xxO3w4)8M}1R-Hzta_ulS;>02@`tNPs
zkUl_t&2R}<^1ypQ(zlSBum?mUR{NuN>zAzYSE||I>LzL%bMpr0({zU;>P^(4O11w+
z4Q0CTUUZ!DFM@mr9v<}M1gOaAHEQ)@5+Q){&eNx_e4+rJODn(r+{?K_#Y9Y0H?lz0
zKv`KCY)E5R@7wWX?%GMm)6>Vthg3#pmpM$`&E0tj$u__6dbeJ`(NBGp`Sx6%5*g8n
z%Wp5C^7z@I>pdYIY<3frX+e5S^hD2hQ#Z$vwsFad4O_(phpV^h-BZPp>3S0rtel5x
z`dN{5Y^BR;!<&hc^gQ%-?=9YGyoB&8E6B;g-7mo^DqmzD1Jz#1v}KL<3PX4U@$zzv
zyF4fb;FL~P{bK%G^rw=@;9y}@R@4BL8~dp|H~U~nXaw9HEv@3{2y_X+;qb;}8z=($
zt5HWd)JK}asJf%8YjCDlX|UP%{I#uqlO-J^W3kxL5Q-IG?;v`Eus2p}-vtRPd2^_y
z5d-1Y@$4N`-bqjw#>bBz5q^i(qXqHD9?0S0jm>(jLC!&lQgk}MtxAjP4~)sC-1V;d
z!otva|Nq6_TSrCRMem~)28g~Af~0hVh%^j>Ak84%NJ)cq4WKAe3ew%((mkLwLw6(H
z4MWb%J^1?m?)Tn5@4D;$ao5j(6kIcF=A6$td+%pI&$EroBa)KX(T-TRZoi}3$jx0C
zva_=*($21IYrN*RSMAZaIp6#{<$FtpIKELhd!it`!{zEKeffr&%~uWRCPdsez7~{8
z9A<%=Bdq*z(S-||B*cb?JJO4~^I+MthrZ}LXeh!hPiAq5I07~~C-h`x5zf~7l$=&M
zZSkW)h^^G-xi{Nqj+1^M>W7vthPmN9ww^1IMSf=rpT)fG64zWUoN`&|)lPxi*L8%^
z2UA8bioePPAwhA*YOJzHAAyM|H4S4!J9N&G@&s90ja;9pbw=;RKS4S>=Ou!S59cu(
z+M}8Ncrk>6+puftgpG|gLkA|ZX*xl7P8qZkWcmo#&XYw@SoNWhu34R?3%qQop?o;a1DIG%A1kLYTr>s*k|rYBmdT=+(O{c7Msc_$-78Y{K>;006N
zOAE$Qji>)%0W;^g3ne8ME_7Vap8d1B_ik5jlE5Rfi4W1fS(H{{j;N%QjTSjTzM0Vm
zmL?M-sHbSvZ8;ekt&5W{bh7IHjMC1Ry@>)af$emmEJd4%%2x^X7JWI7n&D`R+B8(p
zNg2B?JZ#EHGMFO2)@^5)({%N61}|7ho|Hp3$$4>Q`_b;#PNSxu&5|u47JYM;YJt3x
zo*5yy2N)#~-kxgEWxh+Tp;viXOoJf{
zM-_fA-Jn0-7|R!Q?f>9qiyVa$E5IChtR_pnnzKAkD07gQy`im2Yb|f|`iuRZKX-A2
zRi;U6bi)NQ-vxb&;td;ZaZ0St33t+~cjOQ8(`mFTgEopLd!aL{!_VqeWa
z-!fMmk2A2Ky=TPfpygiJs1}c{TAeeS=IU8cH0`$4WWCdPhs%gE6_dxKVh&T-;p<(m
zSqManj47WF3bUYA;&;CM?9NPU;$7eua!CUTwp(P#y%Wa|jAjzKY(=wegw>1B1lv`l
z?uPjPqaWEj;}JmnhWYM-YJRpDbsq_jqyL{0=w*OLRn6z_
z``ImC#`fxB+NV&2wt4SRSa`T;_hpf8yUjv0V>|H35fmb73`>6_8V%f2g5D*9%uibqlvdr@igmEB{Q64~>BgsNLmOQ)DF=Q88K~KUe8NO4a$q#2Q
z+`2vR%?C1aq0t8rN8rzv+U!uzsCsbVe!5eAQXxkjsDzX`IEX$&_Wx9Z;8Kk1BMD{X
zrAJmi3Oetg9naM#<}HeZgd0GXpGtzYvi7T0w&4PoU~D_=aZ5JO+y4F!O-=QFzF&L;
zt~t){dM{P?*2@_iM@Zo@$i?M(s_F+nX4NW45-ZJET2025CbVOxC9;n$)Y6rg7dENHIBP4w=DYjL;x8tRAQ|9$Yq_2!J
ze3Xye;-m)MkI8>9WZ&R$qRw9zeA^M4j*-z~xo4*a@{j4c$3>@>PBu63@bF%TB&IOd
zxt(@g^j!aRSi}$%@Qs|?eVRO^PU+sOq^Of0g9+Kf_wN4FSZ_I6*mp7X29L?5iWl(h
zFM?H6!1Hk%&TZ8**;v8f43dqDha~REOW%6G01-7B$q-eJVih_s%d7Hroy+u(ikD6!
zx7sPipR`D)fqoNK28A*ulW<$@oy0{>e$}b5=&73P3p;4;>ua>d{G6(?%F(F_JJ$pL
zPMT09`@t>#!xpTXh-}{=3|W8U?u^F?MyFP1@K@1{$K2X%p>jKtL^3e{Tt2>Pg^x;Z
z1QcHnCXiWt^#6K!9=}`h
zJV5#{
zMG#0^-n)0tXx0N$#b0)DG`i6e2gW$_zf;R+_JN
z+eCtLBW;$0{f2g>8Q?^WoK~Nbg37XHkqS5o6);hHmU<#?0}A{E`*ALpdXb*G+yaw&
zrPUaFA}ee6C!bd~dE`9&{AVF_2!$kp!g0qRRkj25u|Yv)9jPr?-odgJ#X1#Q@nZQM
zIXN|l$$GI50VCH+GrFULvDhy}ArV4DT4OvR{$w;<CK
z&JQX8v{r)}lC9Qb7hMj`!qm8aj0gwNQ>YO$Ej$+RhFR4|=@=Q8g8lsX{H>ZQw*|oy
ztSVF&0cB7~NmG+(kj=|Z*4NsKPKJOnJzddX-D}yw%t$QJF)HIp8j~}WNXpQjl
z4n$^r-Re~yxTg=nM(2iwpQlpL(TBh>w3H@KwHOzK*m&4?KA9)4X!!Q0cs0av=f%gx
z1-_r(syWxgq4%lr{xLuG5Ico4&
zHD|E_A#N8mRda|wl0}ukAfgeo2*~*PbFa70zr_R2BLED1BPdZ94HM@TQE8!Jad|pK
z`sNE4yH7|itKP@eNI-rG1PE=9^bG0fB3QyTm@Ym&zN1dw|3ef7@_~G01joso$Hif>
z!$^KXcemUf+%^3|3&MSAUq4Cz4=&AFJycYEFb(SabEQrfzeR6mh#@;y^>fuV0~*De
zzdW!*=nZl}AYP!r0YNrh;3;r3Y`7QxU~9$%pm!bxr)W8h#k*kdxfaTAfWfQg=yG^`
zSm%xTxDZ9;<%M=w>G=Z>M5tt@F;eNUx#uXR)~Ba`Qn?F|U$DhN<>fCl8>{z{kOL}r
z{4hx>IzC?eBXSdia|=GO^-{v{S<2%50N`ZT?1ziG3=aXfLIjE$|Lk}I@k7;el!*ku
zpE}4@Y5+4V7XjW~5F;}LBI%HMm2@aY{7bQnQ1_fw+TiBLxK674vi4~
z@_zdqcb|p?RQ5qcy|w^#vLEZwu3~_q#@ol#k2@A7IP_~Ki+VQ0M#agVxh>q1xR^6)
zH}O?7>EdHPfh6zw-_?WyoFXGFP2-}$epx1*A=o?JgI9N^zyq6>o_8zC%%W`8EKOfyFIg;qqYg0hk_%&4}QIg16Ac#
zj0bX?D4S{%oK?(u{yIqb)q0=d`%PAxBV0t#3E%g+qpb%P=+1Dzh}-aaEf8qd)1|dF
zMsYj}&Ht0rG|8rYogdQKan}j6iiX--IBL~K!jnE01vh-uliC={|7{`~`U;^j#>pG?
zEqNXvFENRVFej~h=rbVHt!!>GY+4Qf_*uo(sH96p|Mb}&NMi)v@5Pp8R9ByZM6Ax{
z7#lp<)C;Vk>!6Hv-YFKjatxeUKi(Vo9H30;>dx#T{u%Io-2#!(Ln59hiJ@J(si~=y
z3Ku#Jd|OAcFd{8nl76Tgdc#+3u|0So=AV_1Lffi6L(@9Cs%4`h(d^NLS|dfOTvg9T
zr`k=2Rtigd^=yw$(RIb3xKRv)eE+7xVSA1gyRP423Ypj*oUFJFkcdlk
zUF^;ChTO@Cl2?NJ^TFQUffgg_m_-Kaz+^sygsqqbXw`=uzh<;aol1&=$S%$#%M)
z$^7Twp#A(Me_lyMgfQuD`)*^W-Q};jm7F#uF>pmhqiYm}OKa>Fqg2AgB|96%oSl7-
z3A;k2pvc`>DzQ5f%(VV}4f=BsmT0tru`w-af!7SX3AAtuZWL`1;yN`}q(>#|o3VAF
z@_SK)%Gmu_+
zv1;>5vNYUKI179Z#kqHH=7(KbL7B25jacytbxMJ3aN>_+6IBj%n5uf>F
zg`zDYBO`PVsDon{&v&TRRSJOzWn1-8i0??gYGiVg{6V?t_iUz!93gpqsP}pHP_5H8
ztHR3x8c5dJp%*~!q$emxAI|keIL+=?=j%18e`v`rbF8^k%U&Q-Q!~lz2mOJ4lkci|
zg&@xHj#6nR}maEubL?v
zH{qv1-@4=Xm6)5Jnp&wsxg|Ycot|V|V`|q}k=^L&9Wx}VO25u@u1R*}fH0qyjld&eq{upcOuI^H*TunUd%7mQ
zKsmu8xF#(ipmXY#Uk9`)L}*)N$sC)G4a%YqpC0Gjy(}?9pAEG`$puT_qZ2qF@mcBV
z;fudz2xkxxb`eexH@v5Waf=KJv);dRCx^}})KKThPHWh=jgfQYdgdg}?8cC3Umj8j
zw&b}#Xfkor50{+!LL;(;lGl32<@z^f2-SaBz|{r=<*``)XYn&yJm43a`%gs5W!sFYc+h^L+STS|o*I_ioSuQ1c^>iAf%JsWVE>)H>M#1)7ts`Wbi4DMcQG
zCJ5-tGX&Dhdj_`BZY2GU#RaR=XE=HB*?kG&xH-TjeXKVzV;a}nL&_I>*`r}#F`ol6
zRq2N6*^GYuz*~FJ$5oFQetaY#zh`;`IVso7!`|5uvC-oTD@@7N&@w8-Q
zNs)=g@MxE#?y$KXWdXXPL{)};v>=b||^3#~7jxQDf-9WdsI#_gLVQx#cKI`|Hfd}~CTj^T?*6~q@
zjTf{OHz(3Pmb)Skh|?jjG@Pf}eKOJOUrAEu3)MK91zdJwp_d1d`K8wZ4+Wp0o?+Q$
zuL<2}Co2hd<(kB6eP*ogo^E5|(#$ZNu4RO}aaAp_M6f+{->jz`0SJ3y9a6|6AwP`X
z1hK-s+%eJFxj9cWl4K8lbY~f~-LD=Qm8YbGQv*FDFGXqJ6`rhI{&%W+-T
z6_i8Y+h^mr?WLy0nt!peu|>w0*x6CI?B+hmQLj=6MhxUzOr+HmtZr0n0utdHo4n%O
zS+?j{jl6cDv)y^2+e>_(z^kmcP6JBMjKO(p!7|?9rgJOcS1GcR)}M{PTJ&vx
z|2g{oG_^uam-fXXt7xllB|e#O6^)>ko&zr_0HU{yy{?df!WP^p%U)en1_wWIBO4kX
zF1L;!1x6lWSL+@jm6_JxH7Vnz>~P%Lhx6I5U04(P>GrX`PU?9q=kaEzJ6NqtN|87E
zMLLmNUp|qnZ>|dHF`P_%ms#l2X;6mpQm&?)7&0i<11hJna7l^p0fu%<`0x
zqu8&s7a)-%Ab9^>-_eTd3{eWSrA0)UQjom!=hx7^`vka;;#jXB>uwDvTQeE05N$X>
z1hb;NSJ+)0SOh)jvh)IFBlI*vm(5*-gp6~2I|MdH3lmS$L0gGgy@)ml$7XI{k*%CL
z)=mxnykXiVDTnV9>^}Pga62uTPu|6Ocyi)>xfXf?{7FJf=Bk}!o$H#ZQ_$Slt%2Yq;PL}8XK>V(n3Bl4i-V>Jo=_5}iSr}fSBEEKW4cxQzNoQ|?
zutlAh`n)32#u@R;FgCtlcft3RZrh`=pXA-^JP%war?FASG4z+Jah8S{YQ)g+X4UkJ
zPDSpMJRNC0GeR;l>eQB}P-Or~$peF-&hTu|gv9Lb;0U6*cJ!=5}jX#n2+Rurz<$a>w?
z@ZigNP=57M%_76UUzUO0;t2;|UZQ~%qnp+}v81e*Nm=ZyLYGx%L6ufhp$Q4Bk;I%O
zc1qVlL)+mDGb3R%^1$+)Mc)2hHMOXJYjhcPW-OAtnRsZm*J74RzoX;wuIcO7Umk}b
z3`}-kRMf~b0v0{b_v6zG5wC#2dF+jqui&Zgsw}d3(B94XYq!wfCEM%BJXkjU5ASBr
zV$Ix?YEzTspArbIR?U;>P|l+wZ*m|XBZ5G9|H7i>8IwqFQseux1HussZSAQm#E0-XX~hUSUp;@aL7Rvv
z8PGz|f?1OXyGo-<Bl!Bl75+muy0OF+SbkPk9YZqz9K66fc%celfmw1v
zDpnyK&uBn42ewcaqXqLDT0p8EpPz53Nwhj(1l0EDG7@5_Ax>^?PundsCX5P(_hUuH
zrT3GT_IBC0oH^wy%HGS)H_X_DpFtxH;gi}diM|)&|Dz1_Kq_~eL?`4YfV3Ahc2lNR
z^asYKBoCe5Dk+75s-jfkIF5~g#_r8bn%j`WYVsMj^<+JFI=Q#Y|3kZ~xm
zG6`FJxFsU8b2A^KLny(p2ohpqDL41iz0d;5hLr2A{zCHPZj))PvcvfWfF>d0H5~&Y
z{0knQsKAi(?C_WvzID_-M|E{XW+u!)OkcmJ%J~?%|0mG^mHfi!MY65s+;se0CgEl#tpx`MBJN9R(m5F4QVU%`n7*&_ie2oxNlc|!O;t=9TmwZfR?Pvk{{l%F;S@}M
z*o$N%6>zdjHi}eruU)CzB78cLO*1DC*>x)q0ASwP*%|iuUEjIremu8hM@o^d00Id)K*pYJ
zKfJoau3e)G>A@*3iHKt&|N2#sg1mTN@54tX^Qm8k^}&hwh$RTrGgHvF_YK>=>=RwG}}PnY#(K^zD|Ye2#k&ioxY~
zJk2>W1TmLpli%sQJN&~3Aqr}A@~_7Bkbhedc~`B7uap->?yKYb_#tQqxuI8?0K@4!
zKU3ERZxnYQhawXCKzMt{<4{caH-U1teGYvBV6R
zG*B{n07{!saE;L_MJUbIjLYx#WbY?OqkCG%Bpl}`s5JhoSFcPiRw)}*p=&(0GhTcB
z2LHw)ytaV>QD#mZorFvnvyYH`7Zdi>4|7?H3O*p32*T(vUqRL^G?CY?bFd4iU!&$e
zco7R$tTV6RGq9Ii=0x|Bhfu>m>5*1IO}FPZ58(#Bu5Xkf36-$rJQnq0y6h#z)jT~Z
z=1aJ%G)4QRM|0Mx>*(&4FRDzDl8TvXzFN=EPq#Y+S_u(#i}W!5Mbcx0P-aad+vV>?
z=*R_wf-AJaVQH_*M0m`0I)T?aG$I^y&I6PoVUgE0wKVFvUB7=`H38;zoA>VC55xZT
zBlXpAbPAFBh3EML9ng53rXa+_6YyGy0L`QP-MGD3;@q5i)aYBlgcwPBB^kWps)h}w
z28f`ZOJ;~slWt6=yL_T&ll(^Eay;oY!U^aygpF#Ycf2r*eV_rzPoYd}makSEDJ*-u
z;kL}Q0f+!z@m$)K_H}yza0LDNzl|E&Z=?jR!oB4xjS6f`$$|7x(!3}wS~@y;)SB`{
zoT^N+_kM+YY;VV-W?CVRW3cR6O}T#-_uvH`mjjn(AApmmZwGb5Mz{Et1}sKOC4dFC
ziieMnPqsHV7jU|w!y;lmHyQ)p1Nd9KdqANQf1mggIl3*VdPl|4#d2Ix&yo%y>B2el(0<{PvTCD+u|VY%KGOne=ik+=uyGt`C-Tg3KS|LmoMt<%Eo|va=(<&dwa@T5#|k|?+-vb
z34xr|nAl2WP8(-J=LIIYTveHs+rj*8)2i~h2BN)RJ(1TDm
z?XBfQn8Fp@^(jJtt()sEs`H~ot2MY5$uZ~%V6!r!z^gvCs?k^oPTpdBCs#$mgb}42
zg68<@=D3!!)-X^}7DdB_PSz%vjkGS_(apGI@e%6j9d%y&=d8RL;Md>cpYa0=Jpeff
z>U&?HMz;@^=i_-TCrjP9?)4%1`wq=17kYjQg1Q4{3FRU&0=8&Cy%wmn1v-7zX`m4+@#uqhNS)Cs1H1n@N)feR
zI5|4GX`!j9R;epcmt$~2V*2JX7b>4Q+Ay>C9>aOGPSOd|oU(FIRj9%sZ
z3MV$YZQq`-5f++L$_u+4l!p`-TdY#bMv?QM_iYyH^+2xarHa*n@gW~Ryo7QTgZ^!9^5ExvBZmae
z+yOsjhJb7833(RT9rTaU==KQd-9Pv5wTSz=f))8|E9V{U>Wfzywa3kdweqhW!#Aq+
z<7f%4b(V$HVx>#3N>D%iw=}#N4Y_l*wUxA`kXbF#u(
z8Fa#!BU$|X{6zMSCTp$zPH!M6*cVVX7RMC}F@T_3aHmU~4rl^<=P)IIsuXc%U0*dZAl|RC(&2T!6hEeBU;{_oTH_+RAyL=nrnvJn
zRGImlmx%1B_iC`;`MGKyD;w)dAF5T#x(nw(qQzUc&Qa+hFm>9X*;P#oZ8y!veYED!
zy6C4_W^w`>)&mMFqV5Y-W_tiBIe4F$_xI12`kpjUK=4G|L@LF650vRw$*Q~0(ec_<
zyQgPo`Ofr!3Dw6QxjtNiYC?NvuBH_iH||yO&v?L}kP>B2HohI8q+YU6Y`irS{V(4u
zF5vz=0xAMCy(IWXLrI?*NSWIYJD%|CQ%|5GlumjFdNoQxS2WX}L>z2vy;`fxi0s{W
zy*X<;^z*GpYqRod*J=Eru%6oEz)C<&?`w!7aEp`6?HM{h+Q>z7PkF!88_yWdG57=m
zleB~^On`#o#CXJM{?Q{65%Nb{eW&)dNl8fs`t^q*Fy1oLWU|piL~wzpapH1U6v}x;
z9S|78udWV&IBbr7#Fi)FovbzY1a7mLmz#sRi@}AYWMpIq6v*C=`Nqr3^lG25a(edq
zD%+V;m?>5)Aer)V_Z%aG4Cd#-+%+kVmoT_D8s5|fX6ZF{;xiOzoJ+#tq*yry&F6?{
z)?QoAWLH3J^j>}ZMN3P|W~yE{D$;knjTjl
z?HYDh6TBg5H^~!AD&b}0tgRGB%H4w(p(0?gK`*d-9*pbojzXPz%<+22x*C9{B%&@D
zWH*9?IG|KkwKy!lqVQM^=pcSBICuxg)57^!Sn@0Kqv~qaI>ypllB*lX3an-eq=P*+
zhl{wqcQ^nNMiOFVY*zhtwm8t_^6Dc*$YqxF%jBpGpgKMZ>wE6U78ZVsdiaQ`55U_x
z2kl~(mW2x+uA^qB>%FYO^;7Xg$at>_v`E#9RNjb*UN!cs$$E_9R<@_AGKq$AjV`<&
zQW6VWd`3RZebX@6);I>^a|%-_5Em+OUw*#L@HCi`_uXPGx0_$;LX{C}vXQ<)L=TZ$
z0sv(FItyQ^hzz2kMq5fdM$PQxWX!65_rDSoM$RbqIOWV!L(2W(gV`oPB=2@Dd-DJq
z^dFAB&lPDM%TCq%GWivFZWdFnqYh%es{<}qh)9C_!T3DjOiQ0p75O7~CXtILbLD2U
zFWM)V7k-()#lWdW;*L?geY}Y{HS6Q4GaHRj0WNx;(RVqhlU!0gR2OLQCl+!SB~4*x
zZs4{)kh%}@_xj&e`VY5eVy7aqsOc3J_dd=b(3cG^eH1mBT!6~Fu%N#bnXGW3c*~k3
z#N62UXy8sbhQHkQbjo&_=VTbDMrE^u0Zp{zJ(|qn&?h#Md(+RBMTL=LbGpnH15%KG
zUD;)BJ8+d5-Enp_yQsa=(*$O-H?_XO7KA$3Gjo9_$Gcc_0_Y&P%E%tG@;-lIHMZgu
zT3(=EB@=ItYt}bX@vE|D6JWLR5uS9kC
z{{4gf<3r?rKyYwnMMVWLmL{6lDzZ@1QBqbeo~vepA3mB7xSLL!(3mJI6%TpDR{T-q3=LOfU72p0(
zr1kd~FK_p}bX*B$@g%!&C3Kax3g!}~iZB!b??gq$wo{$JT{O1WDF
z*y;cKL~Ff$!orxJe~MY4UlseSU)^IikYM?r-|5=_lSvl;Z%o7Z-|zAN!)-XF^l-WU
zSYoX8`X&2JSQ=yIIB1N_d^47*xqCP`y4B(l-@ij518&JRU3&>=V+DTMrZRyYg^6i2PxBM1uti-}K|!(Dh8(%-FP`?vFQiDjaoxj=QKS?wbpg!=Y-d>5
zbLU@6mSf6-L(}CZQ>iEqnb>%{@Om%>8HQ?$?CFtXI&H