Skip to content

docs: ADR link fixes (superseded by #9)#8

Closed
adel-saoud wants to merge 1 commit into
mainfrom
chore/strip-narrative-leftovers
Closed

docs: ADR link fixes (superseded by #9)#8
adel-saoud wants to merge 1 commit into
mainfrom
chore/strip-narrative-leftovers

Conversation

@adel-saoud

@adel-saoud adel-saoud commented Jun 17, 2026

Copy link
Copy Markdown
Owner

Superseded by #9.

A pass over the codebase before going public. None of these change
behavior; they all make the repo read better as a finished artefact
rather than an in-progress build.

Source comments:
- `ingestion/pipeline.py`, `retrieval/embedder.py`, `generation/generator.py`,
  and `tests/test_config.py` carried docstrings that referenced "Layer 3",
  "Layer 5", "PR 1", etc. Those were notes for the build process and
  don't mean anything to a reader landing on the public repo. Reworded
  to describe what the code does, without the timeline.

ADR cross-links:
- The seven ADRs used `[[001-name]]` memory-style links that render as
  literal text on GitHub. Replaced with standard `[ADR 001](001-name.md)`
  markdown so the link graph actually works in the browser.

AGENTS.md:
- Companion-project list pointed at local filesystem paths under
  `/Users/adelsaoud/...`. Swapped for GitHub URLs so the agent doc reads
  the same whether it is browsed on GitHub or referenced by a tool that
  cannot resolve those local paths.
@adel-saoud adel-saoud self-assigned this Jun 17, 2026
@adel-saoud adel-saoud changed the title chore: cleanup chore: strip build-narrative leftovers and fix ADR cross-links Jun 17, 2026
@adel-saoud adel-saoud closed this Jun 17, 2026
@adel-saoud
adel-saoud deleted the chore/strip-narrative-leftovers branch June 17, 2026 16:01
@adel-saoud adel-saoud changed the title chore: strip build-narrative leftovers and fix ADR cross-links docs: ADR link fixes (superseded by #9) Jun 17, 2026
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.

1 participant