Skip to content

docs: publish comprehensive master README guide#2

Merged
molhamfetnah merged 34 commits into
mainfrom
feature/master-readme-guidance
May 4, 2026
Merged

docs: publish comprehensive master README guide#2
molhamfetnah merged 34 commits into
mainfrom
feature/master-readme-guidance

Conversation

@molhamfetnah

Copy link
Copy Markdown
Owner

Summary

  • rewrote root README into a full command-driven onboarding + day-2 operations manual
  • hardened setup/operations/troubleshooting commands with repo identity and worktree safety guards
  • aligned README references and implementation plan snippets with existing docs and safe command model

Test Plan

  • bash scripts/tests/test-track-git-status.sh
  • bash scripts/tests/test-git-tracking-shim.sh

molhamfetnah and others added 30 commits May 1, 2026 06:35
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 4, 2026 13:37

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR replaces the minimal root README with a comprehensive, command-driven operations guide for the ROS parent repo (and its program/* submodules), and introduces a “git status tracking” mechanism (collector script + bash shim + installer) with accompanying design/implementation docs and tests.

Changes:

  • Rewrote README.md into an onboarding + day-2 operations manual with troubleshooting and contribution workflow.
  • Added a git-status tracking system (collector, shim snippet, .bashrc installer) and end-to-end bash tests.
  • Added specs/plans under docs/superpowers/ documenting the README and tracking-system design/implementation.

Reviewed changes

Copilot reviewed 10 out of 11 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
.gitignore Ignores .tracking/ artifacts produced by the new tracker.
README.md Full rewrite into an operations manual; includes tracking usage section and references.
scripts/track-git-status.sh Collector that parses porcelain output and writes .tracking/latest.json, history.ndjson, latest.txt with rotation/locking.
scripts/install-git-tracking-shim.sh Installs a snippet source block into ~/.bashrc (idempotent).
session/docs/snippets/git-tracking-shim.bash Bash git() function shim that runs the collector for status/st in the tracked repo root.
scripts/tests/test-track-git-status.sh Tests collector output, rotation behavior, collision handling, and concurrent execution.
scripts/tests/test-git-tracking-shim.sh Tests shim routing/guardrails and installer idempotence.
docs/superpowers/specs/2026-05-03-master-readme-guidance-design.md Design spec for the new master README content/structure.
docs/superpowers/specs/2026-05-01-git-status-tracking-design.md Design spec for the git status tracking system and constraints.
docs/superpowers/plans/2026-05-03-master-readme-guidance-implementation.md Implementation plan describing how the README rewrite was produced/validated.
docs/superpowers/plans/2026-05-02-git-status-tracking-implementation.md Implementation plan for the tracking collector/shim/installer/tests.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md Outdated
Comment on lines +88 to +91
real_git="$(_git_tracking_resolve_real_git_bin)"
tracker="${TRACKER_SCRIPT:-/mnt/data/ros/scripts/track-git-status.sh}"
tracked_root="/mnt/data/ros"
cmd="$(_git_tracking_effective_command "$@")"

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot apply changes based on this feedback

Comment on lines +100 to +104
tracking_dir="$top/.tracking"
branch="$("$real_git" -C "$top" branch --show-current 2>/dev/null || printf 'unknown')"
porcelain_file="$tracking_dir/.git-status-porcelain.$$.$RANDOM"

mkdir -p "$tracking_dir"

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot apply changes based on this feedback

Comment thread scripts/track-git-status.sh Outdated
Comment thread scripts/track-git-status.sh
Comment thread scripts/install-git-tracking-shim.sh Outdated
Comment thread README.md
Comment on lines +140 to +144
cd <path-to-your-ROS-clone> || exit 1
git rev-parse --show-toplevel
git status
ls -la .tracking
```

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot apply changes based on this feedback

molhamfetnah and others added 4 commits May 4, 2026 17:20
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@molhamfetnah

Copy link
Copy Markdown
Owner Author

@copilot apply changes based on the comments in this thread

@molhamfetnah molhamfetnah merged commit 3da07d4 into main May 4, 2026
1 check failed
Copilot stopped work on behalf of molhamfetnah due to an error May 4, 2026 14:23
@molhamfetnah

Copy link
Copy Markdown
Owner Author

https://gh.io/copilot-coding-agent-docs apply changes based on the comments in #2 (review)

@molhamfetnah

Copy link
Copy Markdown
Owner Author

@copilot apply changes based on the comments in #2 (review)

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