Skip to content

chore: remove pr.md stray draft accidentally committed in #997 - #1341

Merged
jdx merged 1 commit into
jdx:mainfrom
sahidvelji:chore/remove-stray-pr-md
Sep 6, 2026
Merged

jdx merged 1 commit into
jdx:mainfrom
sahidvelji:chore/remove-stray-pr-md

Conversation

@sahidvelji

@sahidvelji sahidvelji commented Sep 6, 2026 •

Copy link
Copy Markdown
Contributor

What changed

Deletes pr.md from the repository root.

Why

pr.md was swept into 13466dd (#997, "feat(builtins): use types = List(\"text\") instead of glob = \"**/*\" for text-only hooks") alongside that PR's actual changes. It is a PR description draft, and its contents describe a completely different change β€” test(setup): exempt nanoid from aube trust-downgrade policy β€” right down to a generated-with footer.

πŸ€– Generated with Claude Code

AI-assisted β€” Tool: Claude Code; model: anthropic/claude-opus-5; version: 2.1.236.

Summary by CodeRabbit

  • Documentation
    • Removed documentation describing a CI compatibility fix involving package trust policies and a transitive dependency.
    • Removed the related rationale, configuration details, and verification notes.

`pr.md` was swept into 13466dd ("feat(builtins): use `types = List("text")`
instead of `glob = "**/*"` for text-only hooks", jdx#997) alongside unrelated
changes. Its contents are the PR description for a different change entirely
(`test(setup): exempt nanoid from aube trust-downgrade policy`), complete with
a generated-with footer.

Nothing in the repo, docs, scripts, or CI references it.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 6, 2026 •

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. πŸŽ‰

ℹ️ Recent review info
βš™οΈ Run configuration

Configuration used: Repository YAML (base), Central YAML (inherited), Organization UI (inherited)

Review profile: CHILL

Plan: Team

Run ID: b8587cfd-ad7b-4dee-ada6-fee8db7dc2fc

πŸ“₯ Commits

Reviewing files that changed from the base of the PR and between b2e99a2 and 52921e7.

πŸ“’ Files selected for processing (1)
  • pr.md
πŸ’€ Files with no reviewable changes (1)
  • pr.md

Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review.


πŸ“ Walkthrough

Walkthrough

The pull request deletes all content from pr.md. The removed content described an aube trust policy failure involving nanoid and the related test configuration fix.

Changes

Cohort / File(s) Summary
Documentation
pr.md
Deletes the CI failure problem statement, fix rationale, configuration change, and verification notes.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: βšͺ Minimal Β· up to 52921

This change removes an unrelated stray pull request draft and does not alter repository behavior. No current merge-readiness risk remains.

πŸš₯ Pre-merge checks | βœ… 5
βœ… Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage βœ… Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check βœ… Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check βœ… Passed Check skipped because no linked issues were found for this pull request.
Description Check βœ… Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check βœ… Passed The title clearly and concisely describes the main change: deleting the accidentally committed pr.md draft from PR #997.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❀️ Share

Comment @coderabbitai help to get the list of available commands.

@sahidvelji
sahidvelji marked this pull request as ready for review September 6, 2026 18:46
@greptile-apps

greptile-apps Bot commented Sep 6, 2026

Copy link
Copy Markdown

Greptile Summary

Removes an accidentally committed, unrelated PR-description draft from the repository root.

  • Deletes the unreferenced pr.md file.
  • Does not alter application code, configuration, tests, or runtime behavior.

Confidence Score: 5/5

The PR appears safe to merge because it only removes an unreferenced stray draft.

No actionable failures or repository-rule violations were identified, and no tracked tooling, documentation, tests, or workflows depend on the deleted file.

Important Files Changed

Filename Overview
pr.md Deletes a stray, unreferenced PR-description draft that was unrelated to the repository’s implementation.

Reviews (1): Last reviewed commit: "chore: remove pr.md stray draft accident..." | Re-trigger Greptile

@jdx
jdx merged commit f51837d into jdx:main Sep 6, 2026
26 checks passed
@jdx jdx mentioned this pull request Sep 6, 2026
jdx added a commit that referenced this pull request Sep 13, 2026
### πŸš€ Features

- **(builtins)** add destroyed_symlinks, forbid_submodules, and shebang
checks by [@sassdavid](https://github.com/sassdavid) in
[#1360](#1360)
- **(builtins)** add mado linter by
[@hituzi-no-sippo](https://github.com/hituzi-no-sippo) in
[#1349](#1349)
- **(builtins)** add terragrunt and kubeconform by
[@sassdavid](https://github.com/sassdavid) in
[#1363](#1363)
- **(builtins)** add terraform_docs and terraform_validate by
[@sassdavid](https://github.com/sassdavid) in
[#1362](#1362)
- **(config)** **breaking** add shared top-level steps by
[@jdx](https://github.com/jdx) in
[#1255](#1255)
- **(hook)** **breaking** make staging contextual by
[@jdx](https://github.com/jdx) in
[#1256](#1256)
- ship hk configuration and debugging skills with packslip by
[@jdx](https://github.com/jdx) in
[#1337](#1337)

### πŸ› Bug Fixes

- **(builtins)** run tflint in each module directory by
[@sassdavid](https://github.com/sassdavid) in
[#1361](#1361)
- **(builtins)** make zizmor tests pass with hash pinning by
[@nettlesh](https://github.com/nettlesh) in
[#1366](#1366)
- **(builtins)** use --format files in rubocop check_list_files by
[@andyw8](https://github.com/andyw8) in
[#1355](#1355)

### 🚜 Refactor

- **(builtins)** **breaking** use flat configurable steps by
[@jdx](https://github.com/jdx) in
[#1253](#1253)
- **(config)** **breaking** remove deprecated v1 interfaces by
[@jdx](https://github.com/jdx) in
[#1257](#1257)

### πŸ“š Documentation

- overhaul guides and redesign the documentation site by
[@jdx](https://github.com/jdx) in
[#1340](#1340)
- document escaping literal braces in step commands by
[@sassdavid](https://github.com/sassdavid) in
[#1359](#1359)
- write PR titles and descriptions for release notes by
[@jdx](https://github.com/jdx) in
[#1364](#1364)

### πŸ” Other Changes

- **(ci)** use self-repository workflow references by
[@jdx](https://github.com/jdx) in
[#1353](#1353)
- **(release)** publish native completions in packslip by
[@jdx](https://github.com/jdx) in
[#1336](#1336)
- remove pr.md stray draft accidentally committed in #997 by
[@sahidvelji](https://github.com/sahidvelji) in
[#1341](#1341)
- cache perf source downloads and install only benchmark tools by
[@jdx](https://github.com/jdx) in
[#1343](#1343)
- clarify matrix job names by [@jdx](https://github.com/jdx) in
[#1368](#1368)

### πŸ“¦οΈ Dependency Updates

- bump mr-boxington to 1.8.3 by [@jdx](https://github.com/jdx) in
[#1339](#1339)
- bump mbx to 1.9.0 for performance builds by
[@jdx](https://github.com/jdx) in
[#1342](#1342)
- lock file maintenance by
[@renovate[bot]](https://github.com/renovate[bot]) in
[#1345](#1345)
- update zizmorcore/zizmor-action action to v0.6.3 by
[@renovate[bot]](https://github.com/renovate[bot]) in
[#1347](#1347)
- update anthropics/claude-code-action action to v1.0.211 by
[@renovate[bot]](https://github.com/renovate[bot]) in
[#1346](#1346)
- update jdx/renovate-config digest to 8cabc2e by
[@renovate[bot]](https://github.com/renovate[bot]) in
[#1348](#1348)
- update anthropics/claude-code-action action to v1.0.213 by
[@renovate[bot]](https://github.com/renovate[bot]) in
[#1350](#1350)
- update communique to v1.4.0 by [@jdx](https://github.com/jdx) in
[#1365](#1365)

### New Contributors

- @sassdavid made their first contribution in
[#1362](#1362)
- @nettlesh made their first contribution in
[#1366](#1366)

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Medium Risk**
> Major version tag ships documented breaking changes (config, staging,
v1 API removal); this PR is mostly version/changelog/docs pins but
signals consumers must migrate.
> 
> **Overview**
> **Cuts the v2.0.0 release** by bumping the crate and CLI metadata from
`1.58.1` to `2.0.0` (`Cargo.toml`, `Cargo.lock`, `hk.usage.kdl`,
generated CLI docs) and adding a full **2.0.0** section to
`CHANGELOG.md` that records the major release (breaking
config/hook/builtin refactors, new linters, docs overhaul, and
dependency/CI updates).
> 
> In-repo examples and documentation are updated so every `amends` /
`import` package URL points at **`v2.0.0`** instead of `v1.58.1`
(README, guides, public example `.pkl` files, `hk-example.pkl`, and the
commented pin in `hk.pkl`). No runtime logic changes appear in this diff
beyond versioning and release notes alignment.
> 
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
5a74e40. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

---------

Co-authored-by: mise-en-dev <123107610+mise-en-dev@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants