-
Notifications
You must be signed in to change notification settings - Fork 0
fix(docs): render the repository-structure tree as a tree #23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -1,43 +1,59 @@ | ||||||||||||||||||||
| # Clone the repository | ||||||||||||||||||||
|
|
||||||||||||||||||||
| git clone <https://github.com/hyperpolymath/pons.git> cd pons | ||||||||||||||||||||
| git clone https://github.com/hyperpolymath/pons.git | ||||||||||||||||||||
| cd pons-asinorum | ||||||||||||||||||||
|
|
||||||||||||||||||||
| # Using Nix (recommended for reproducibility) | ||||||||||||||||||||
| # Using Guix (recommended for reproducibility) | ||||||||||||||||||||
|
|
||||||||||||||||||||
| nix develop | ||||||||||||||||||||
| guix develop | ||||||||||||||||||||
|
|
||||||||||||||||||||
| # Or using toolbox/distrobox | ||||||||||||||||||||
|
Comment on lines
7
to
10
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win 🔎 Supported by static analysis🏁 Script executed: sed -n '1,22p' .github/CONTRIBUTING.md
find . -maxdepth 3 -type f \( -name 'guix.scm' -o -name 'manifest.scm' -o -name 'channels.scm' -o -name 'flake.nix' -o -name 'shell.nix' -o -name 'devenv.nix' \) -printRepository: hyperpolymath/pons-asinorum Length of output: 556 🏁 Script executed: #!/bin/bash
set -u
printf '%s\n' '--- tracked top-level files ---'
git ls-files | awk -F/ 'NF==1 {print}'
printf '%s\n' '--- setup/environment references ---'
rg -n -i --hidden --glob '!.git' 'guix|manifest\.scm|channels\.scm|flake\.nix|shell\.nix|devenv|toolbox|distrobox|dev(elopment)? environment|just (check|test)|cargo check|mix compile' .
printf '%s\n' '--- candidate environment files ---'
find . -maxdepth 3 -type f \( -iname '*guix*' -o -iname '*manifest*' -o -iname '*channel*' -o -iname '*flake*' -o -iname '*shell*nix*' -o -iname '*devenv*' -o -iname 'Justfile' -o -iname 'Makefile' \) -printRepository: hyperpolymath/pons-asinorum Length of output: 4072 🏁 Script executed: #!/bin/bash
set -u
printf '%s\n' '--- contribution guide ---'
sed -n '1,65p' .github/CONTRIBUTING.md
printf '%s\n' '--- repository Justfile ---'
sed -n '1,180p' Justfile
printf '%s\n' '--- README setup references ---'
rg -n -i -A5 -B5 'setup|install|toolchain|cargo|mise|just' README.adoc Cargo.toml mise.toml
printf '%s\n' '--- lifecycle script ---'
sed -n '1,35p' .machine_readable/scripts/lifecycle/install-tools.sh
printf '%s\n' '--- tracked Guix-related paths ---'
git ls-files | rg -i '(^|/)(guix\.scm|manifest\.scm|channels\.scm|flake\.nix|shell\.nix|devenv\.nix)$|guix|nix'Repository: hyperpolymath/pons-asinorum Length of output: 6949 🏁 Script executed: #!/bin/bash
set -u
printf '%s\n' '--- mise tool definition ---'
sed -n '1,35p' mise.toml
printf '%s\n' '--- README full setup-related content ---'
rg -n -i -A8 -B3 'install|setup|develop|development|mise|toolchain|cargo|just' README.adoc .github/CONTRIBUTING.md
printf '%s\n' '--- repository environment files ---'
git ls-files | rg '(^|/)(Containerfile|Dockerfile|\.tool-versions|mise\.toml|guix\.scm|manifest\.scm|flake\.nix|shell\.nix|devenv\.nix|Justfile)$'Repository: hyperpolymath/pons-asinorum Length of output: 4136 🌐 Web query:
💡 Result: <search_synthesis> <source_evidence> Citations:
🌐 Web query:
💡 Result: <search_synthesis> <source_evidence> Citations:
Replace the unsupported Guix command with the repository toolchain setup.
Suggested change- guix develop
+ mise install📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||||||
|
|
||||||||||||||||||||
| toolbox create pons-dev toolbox enter pons-dev \# Install dependencies | ||||||||||||||||||||
| manually | ||||||||||||||||||||
| toolbox create pons-dev | ||||||||||||||||||||
| toolbox enter pons-dev | ||||||||||||||||||||
| # Install dependencies manually | ||||||||||||||||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win 🔎 Supported by static analysis🏁 Script executed: sed -n '1,25p' .github/CONTRIBUTING.md
find . -maxdepth 2 -type f \( -name 'guix.scm' -o -name 'manifest.scm' -o -name 'Containerfile' -o -name 'Dockerfile' \) -print
sed -n '1,80p' JustfileRepository: hyperpolymath/pons-asinorum Length of output: 920 🏁 Script executed: set -eu
printf '%s\n' '--- environment-related tracked files ---'
git ls-files | grep -Ei '(^|/)(flake\.nix|guix\.scm|manifest\.scm|\.toolboxrc|toolbox|distrobox|Containerfile|Dockerfile|compose|devcontainer|Justfile|README|CONTRIBUTING|flake\.lock|\.github/workflows)([^/]*)$' || true
printf '%s\n' '--- top-level files ---'
git ls-files | awk -F/ 'NF==1 {print}'
printf '%s\n' '--- setup/dependency references ---'
rg -n -i --glob '!target/**' --glob '!node_modules/**' '(guix|toolbox|distrobox|cargo (build|check|test)|rustup|rust-toolchain|dependencies|install)' .github README* 2>/dev/null || true
printf '%s\n' '--- all environment manifests by name ---'
find . -type f \( -name 'flake.nix' -o -name 'flake.lock' -o -name 'guix.scm' -o -name 'manifest.scm' -o -name '.toolboxrc' -o -name 'Containerfile' -o -name 'Dockerfile' -o -name 'rust-toolchain' -o -name 'rust-toolchain.toml' -o -name 'devcontainer.json' \) -print
printf '%s\n' '--- relevant file contents ---'
for f in .github/CONTRIBUTING.md Justfile flake.nix flake.lock guix.scm manifest.scm .toolboxrc Containerfile Dockerfile rust-toolchain rust-toolchain.toml .devcontainer/devcontainer.json; do
if [ -f "$f" ]; then
echo "### $f"
sed -n '1,220p' "$f"
fi
doneRepository: hyperpolymath/pons-asinorum Length of output: 6060 🏁 Script executed: set -eu
for f in Cargo.toml mise.toml README.adoc .github/workflows/rust-ci.yml; do
if [ -f "$f" ]; then
echo "### $f"
cat -n "$f" | sed -n '1,220p'
fi
done
printf '%s\n' '--- setup-oriented README references ---'
rg -n -i -C 3 '(install|setup|toolchain|rustup|mise|cargo|just|guix|toolbox|develop)' README.adoc .github/CONTRIBUTING.md .github/workflows 2>/dev/null || trueRepository: hyperpolymath/pons-asinorum Length of output: 13645 Run the repository tool setup in the Toolbox path.
🧰 Tools🪛 markdownlint-cli2 (0.23.2)[warning] 14-14: Headings should be surrounded by blank lines (MD022, blanks-around-headings) 🤖 Prompt for AI Agents |
||||||||||||||||||||
|
|
||||||||||||||||||||
| # Verify setup | ||||||||||||||||||||
|
|
||||||||||||||||||||
| just check \# or: cargo check / mix compile / etc. just test \# Run test | ||||||||||||||||||||
| suite | ||||||||||||||||||||
|
|
||||||||||||||||||||
|
|
||||||||||||||||||||
| ### Repository Structure | ||||||||||||||||||||
|
|
||||||||||||||||||||
| pons/ ├── src/ \# Source code (Perimeter 1-2) ├── lib/ \# Library code | ||||||||||||||||||||
| (Perimeter 1-2) ├── extensions/ \# Extensions (Perimeter 2) ├── plugins/ | ||||||||||||||||||||
| \# Plugins (Perimeter 2) ├── tools/ \# Tooling (Perimeter 2) ├── docs/ | ||||||||||||||||||||
| \# Documentation (Perimeter 3) │ ├── architecture/ \# ADRs, specs | ||||||||||||||||||||
| (Perimeter 2) │ └── proposals/ \# RFCs (Perimeter 3) ├── examples/ \# | ||||||||||||||||||||
| Examples (Perimeter 3) ├── spec/ \# Spec tests (Perimeter 3) ├── tests/ | ||||||||||||||||||||
| \# Test suite (Perimeter 2-3) ├── .well-known/ \# Protocol files | ||||||||||||||||||||
| (Perimeter 1-3) ├── .github/ \# GitHub config (Perimeter 1) │ ├── | ||||||||||||||||||||
| ISSUE_TEMPLATE/ │ └── workflows/ ├── CHANGELOG.md ├── CODE_OF_CONDUCT.md | ||||||||||||||||||||
| ├── CONTRIBUTING.md \# This file ├── GOVERNANCE.md ├── LICENSE ├── | ||||||||||||||||||||
| MAINTAINERS.md ├── README.adoc ├── SECURITY.md ├── flake.nix \# Nix | ||||||||||||||||||||
| flake (Perimeter 1) └── Justfile \# Task runner (Perimeter 1) | ||||||||||||||||||||
|
|
||||||||||||||||||||
| just check # or: cargo check / mix compile / etc. | ||||||||||||||||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win 🔎 Supported by static analysis🏁 Script executed: sed -n '1,60p' Justfile
sed -n '1,70p' .github/workflows/rust-ci.yml
sed -n '12,22p' .github/CONTRIBUTING.mdRepository: hyperpolymath/pons-asinorum Length of output: 1224 🏁 Script executed: sed -n '1,100p' Cargo.toml
printf '\nMix-related tracked files:\n'
git ls-files | rg '(^|/)(mix.exs|mix.lock|config/|lib/)' || true
printf '\nValidation command references:\n'
rg -n --glob '!target/**' 'mix compile|cargo check|just check' .Repository: hyperpolymath/pons-asinorum Length of output: 660 Do not present partial checks as equivalent to
Suggested replacement-just check # or: cargo check / mix compile / etc.
+just check # Run formatting, Clippy, and tests.📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||||||
| just test # Run test suite | ||||||||||||||||||||
|
Comment on lines
+3
to
+19
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win 🔎 Supported by static analysis🏁 Script executed: nl -ba .github/CONTRIBUTING.md | sed -n '1,24p;106,122p'Repository: hyperpolymath/pons-asinorum Length of output: 1234 Use fenced code blocks for both examples. The setup commands and shell comments through line 19 are not in a code block. Markdown therefore renders them as headings or ordinary text. The commit-message example is also split: the introduction at line 112 is an indented code block, while lines 114-121 render as ordinary text. Dedent the introduction and fence the complete commit-message example, including the optional body and footer lines. Fence the setup content through line 19. 🧰 Tools🪛 markdownlint-cli2 (0.23.2)[warning] 14-14: Headings should be surrounded by blank lines (MD022, blanks-around-headings) 🤖 Prompt for AI Agents |
||||||||||||||||||||
|
|
||||||||||||||||||||
| ### Repository Structure | ||||||||||||||||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win 🔎 Supported by static analysis🏁 Script executed: sed -n '1,100p' .github/CONTRIBUTING.md
find . -maxdepth 3 -type f \( -iname '*markdownlint*' -o -name '.remarkrc*' -o -name 'package.json' \) -print
rg -n 'MD001|heading.*increment|markdownlint|remark' .github . 2>/dev/nullRepository: hyperpolymath/pons-asinorum Length of output: 2914 🏁 Script executed: printf '%s\n' '--- headings ---'
rg -n '^#{1,6}[[:space:]]+' .github/CONTRIBUTING.md
printf '%s\n' '--- tracked lint/config candidates ---'
git ls-files | grep -Ei '(^|/)(\.markdownlint[^/]*|markdownlint[^/]*|\.remarkrc[^/]*|package\.json|\.prettierrc[^/]*|vale[^/]*|\.vale[^/]*)$' || true
printf '%s\n' '--- workflow/documentation lint references ---'
rg -n -i 'markdownlint|remark|vale|heading.?increment|MD001' .github/workflows .github 2>/dev/null || trueRepository: hyperpolymath/pons-asinorum Length of output: 688 Use a level-two heading for
🧰 Tools🪛 markdownlint-cli2 (0.23.2)[warning] 21-21: Heading levels should only increment by one level at a time (MD001, heading-increment) 🤖 Prompt for AI Agents |
||||||||||||||||||||
|
|
||||||||||||||||||||
| ```text | ||||||||||||||||||||
| pons-asinorum/ | ||||||||||||||||||||
| ├── src/ # Source code (Perimeter 1-2) | ||||||||||||||||||||
| ├── lib/ # Library code (Perimeter 1-2) | ||||||||||||||||||||
| ├── extensions/ # Extensions (Perimeter 2) | ||||||||||||||||||||
| ├── plugins/ # Plugins (Perimeter 2) | ||||||||||||||||||||
| ├── tools/ # Tooling (Perimeter 2) | ||||||||||||||||||||
| ├── docs/ # Documentation (Perimeter 3) | ||||||||||||||||||||
| │ ├── architecture/ # ADRs, specs (Perimeter 2) | ||||||||||||||||||||
| │ └── proposals/ # RFCs (Perimeter 3) | ||||||||||||||||||||
| ├── examples/ # Examples (Perimeter 3) | ||||||||||||||||||||
| ├── spec/ # Spec tests (Perimeter 3) | ||||||||||||||||||||
| ├── tests/ # Test suite (Perimeter 2-3) | ||||||||||||||||||||
| ├── .well-known/ # Protocol files (Perimeter 1-3) | ||||||||||||||||||||
| ├── .github/ # GitHub config (Perimeter 1) | ||||||||||||||||||||
| │ ├── CONTRIBUTING.md # This file | ||||||||||||||||||||
| │ ├── ISSUE_TEMPLATE/ | ||||||||||||||||||||
| │ └── workflows/ | ||||||||||||||||||||
| ├── CHANGELOG.md | ||||||||||||||||||||
| ├── CODE_OF_CONDUCT.md | ||||||||||||||||||||
| ├── GOVERNANCE.md | ||||||||||||||||||||
| ├── LICENSE | ||||||||||||||||||||
| ├── MAINTAINERS.md | ||||||||||||||||||||
| ├── README.adoc | ||||||||||||||||||||
| ├── SECURITY.md | ||||||||||||||||||||
| ├── flake.nix # Nix flake (Perimeter 1) | ||||||||||||||||||||
| └── Justfile # Task runner (Perimeter 1) | ||||||||||||||||||||
| ``` | ||||||||||||||||||||
|
|
||||||||||||||||||||
| --- | ||||||||||||||||||||
|
|
||||||||||||||||||||
|
Comment on lines
+21
to
53
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win 🔎 Supported by static analysis🏁 Script executed: nl -ba .github/CONTRIBUTING.md | sed -n '21,55p'
find . -maxdepth 2 -mindepth 1 -print | sort
sed -n '1,30p' Cargo.tomlRepository: hyperpolymath/pons-asinorum Length of output: 3560 Update the repository structure tree to match the checkout. The tree lists non-existent paths such as At minimum, replace the source entries with the workspace layout: Suggested correction-├── src/ # Source code (Perimeter 1-2)
-├── lib/ # Library code (Perimeter 1-2)
+├── crates/ # Cargo workspace crates
+│ ├── pons-core/
+│ ├── pons-protocols/
+│ ├── pons-rules/
+│ └── pons-cli/Update the remaining entries in the block to match the current checkout. 🧰 Tools🪛 markdownlint-cli2 (0.23.2)[warning] 21-21: Heading levels should only increment by one level at a time (MD001, heading-increment) [warning] 52-52: Code block style (MD046, code-block-style) 🤖 Prompt for AI Agents |
||||||||||||||||||||
| ## How to Contribute | ||||||||||||||||||||
| ## How to Contribute | ||||||||||||||||||||
|
|
||||||||||||||||||||
| ### Reporting Bugs | ||||||||||||||||||||
| ### Reporting Bugs | ||||||||||||||||||||
|
|
||||||||||||||||||||
| **Before reporting**: | ||||||||||||||||||||
| 1. Search existing issues | ||||||||||||||||||||
|
|
@@ -54,7 +70,7 @@ flake (Perimeter 1) └── Justfile \# Task runner (Perimeter 1) | |||||||||||||||||||
| - Expected vs actual behaviour | ||||||||||||||||||||
| - Logs, screenshots, or minimal reproduction | ||||||||||||||||||||
|
|
||||||||||||||||||||
| ### Suggesting Features | ||||||||||||||||||||
| ### Suggesting Features | ||||||||||||||||||||
|
|
||||||||||||||||||||
| **Before suggesting**: | ||||||||||||||||||||
| 1. Check the [roadmap](ROADMAP.md) if available | ||||||||||||||||||||
|
|
@@ -70,7 +86,7 @@ flake (Perimeter 1) └── Justfile \# Task runner (Perimeter 1) | |||||||||||||||||||
| - Alternatives considered | ||||||||||||||||||||
| - Which perimeter this affects | ||||||||||||||||||||
|
|
||||||||||||||||||||
| ### Your First Contribution | ||||||||||||||||||||
| ### Your First Contribution | ||||||||||||||||||||
|
|
||||||||||||||||||||
| Look for issues labelled: | ||||||||||||||||||||
|
|
||||||||||||||||||||
|
|
@@ -81,22 +97,25 @@ flake (Perimeter 1) └── Justfile \# Task runner (Perimeter 1) | |||||||||||||||||||
|
|
||||||||||||||||||||
| --- | ||||||||||||||||||||
|
|
||||||||||||||||||||
| ## Development Workflow | ||||||||||||||||||||
| ## Development Workflow | ||||||||||||||||||||
|
|
||||||||||||||||||||
| ### Branch Naming | ||||||||||||||||||||
| ### Branch Naming | ||||||||||||||||||||
|
|
||||||||||||||||||||
| docs/short-description \# Documentation (P3) test/what-added \# Test | ||||||||||||||||||||
| additions (P3) feat/short-description \# New features (P2) | ||||||||||||||||||||
| fix/issue-number-description \# Bug fixes (P2) refactor/what-changed \# | ||||||||||||||||||||
| Code improvements (P2) security/what-fixed \# Security fixes (P1-2) | ||||||||||||||||||||
| docs/short-description # Documentation (P3) test/what-added # Test | ||||||||||||||||||||
| additions (P3) feat/short-description # New features (P2) | ||||||||||||||||||||
| fix/issue-number-description # Bug fixes (P2) refactor/what-changed # | ||||||||||||||||||||
| Code improvements (P2) security/what-fixed # Security fixes (P1-2) | ||||||||||||||||||||
|
Comment on lines
+104
to
+107
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win Put each branch convention on its own list item. The entries remain hard-wrapped into one paragraph. For example, 🤖 Prompt for AI Agents |
||||||||||||||||||||
|
|
||||||||||||||||||||
|
|
||||||||||||||||||||
| ### Commit Messages | ||||||||||||||||||||
| ### Commit Messages | ||||||||||||||||||||
|
|
||||||||||||||||||||
| We follow [Conventional Commits](https://www.conventionalcommits.org/): | ||||||||||||||||||||
|
|
||||||||||||||||||||
| (): | ||||||||||||||||||||
| type(scope): description | ||||||||||||||||||||
|
|
||||||||||||||||||||
| Body: what changed and why. | ||||||||||||||||||||
|
|
||||||||||||||||||||
| Footer: issue reference, e.g. Closes #123 | ||||||||||||||||||||
| \[optional body\] | ||||||||||||||||||||
|
|
||||||||||||||||||||
| \[optional footer\] | ||||||||||||||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎯 Functional Correctness | 🟠 Major | ⚡ Quick win
Make the clone target match the directory change.
git clone https://github.com/hyperpolymath/pons.gitcreatespons, but the next command enterspons-asinorum. A contributor following these commands gets a directory-not-found error.Suggested fix
📝 Committable suggestion
🤖 Prompt for AI Agents