Skip to content

Improve lab tasks and fill in wiki content#116

Merged
deemp merged 26 commits into
mainfrom
improve-lab
Mar 3, 2026
Merged

Improve lab tasks and fill in wiki content#116
deemp merged 26 commits into
mainfrom
improve-lab

Conversation

@deemp
Copy link
Copy Markdown
Contributor

@deemp deemp commented Mar 3, 2026

Summary

  • Fill in empty sections across wiki pages (environments.md, postgresql.md, operating-system.md, docker.md, api.md)
  • Add new linux-distros.md wiki page and reorganize linux.md structure
  • Rework task-2 with e2e boundary tests, log-inspection step, and clearer AI prompts
  • Fix broken links, cross-references, and convention violations across wiki and tasks
  • Update review/fix-via-conventions skills to handle TODOs and empty sections

Checklist

  • I made this PR to the main branch of my fork (NOT the course instructors' repo).
  • I see base: main <- compare: <branch> above the PR title.
  • I edited the line - Closes #<issue-number>.
  • I wrote clear commit messages.
  • I reviewed my own diff before requesting review.
  • I understand the changes I'm submitting.

deemp and others added 26 commits March 3, 2026 13:16
- Update links in lab tasks, caddy, docker-compose-yml,
  dotenv-docker-secret, dotenv-tests-e2e-secret, http-auth,
  and swagger to reference api.md instead of web-development.md

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add User and Group sections to operating-system.md
- Replace linux.md Groups body with a cross-reference
- Add cross-reference to operating-system.md#user in linux.md Users
- Update file-system.md and shell.md links from linux.md#users
  to operating-system.md#user

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- review-via-conventions now scans for <!-- TODO --> comments and
  empty headings, reporting them in dedicated output sections
- fix-via-conventions handles empty sections (adds TODO marker if
  missing) and lists TODOs as skipped in the summary

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add missing "What is" section required by wiki convention
- Split compound git add + git commit step into two separate steps
- Restructure "Undo commits" bare paragraphs into numbered steps
- Add mini-ToCs for "Publish the branch", "Push more commits", and
  "Stage using the Source Control" sections
- Link <repo-name> and first-use <branch>/<remote> placeholders
- Fix <branch name> placeholder to <branch>
- Add missing periods after GitLens: Pull commands and inline steps
- Fix lowercase `git` to `Git`
- Fix typo "It's looks" to "It looks"

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Fix section numbers
- Fix Part A mini-ToC: wrong step numbers and anchors, missing
  entries for 1.3.1, 1.3.6, and 1.3.7
- Fix Part B mini-ToC: missing 1.4.1 entry
- Fix cross-references in 1.4.5 (#135/#136#137/#138)
- Fix testing.md anchor (#end-to-end-tests → #end-to-end-test)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Replace "response is a list" test with a test that verifies response
  items contain expected fields (id, item_id, created_at)
- Add bridging note connecting unit test and e2e test bug types

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Expand section 1.4.3 from two to three e2e tests
- Add test_get_interactions_filter_includes_boundary, which calls
  GET /interactions/?max_item_id=1 and asserts the boundary item
  is included, closing the gap where the <= fix was never verified
  at the API level
- Update failing output, passing count, ToC, and mini-ToC

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Require # KEPT/FIXED/DISCARDED comments in section 1.5.2 to
  make curation decisions verifiable and the learning intentional
- Add matching acceptance criterion for the comment requirement

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Insert 1.4.4 "Diagnose the bug": students run docker compose logs
  to see the pydantic traceback before being pointed to the file
- Move the file path into the hint of 1.4.5 so students reason from
  the traceback first
- Renumber old 1.4.4–1.4.6 to 1.4.5–1.4.7 throughout both ToCs

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Specify exact source and test file paths in the agent prompt so
  students know where to read and where to write
- Add a step to open the generated file in VS Code before reviewing
- Fix step numbering after inserting the new open-file step
- Drop the instruction to open the agent in a specific directory
  (the prompt now carries all necessary context)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Complete "What is an environment" definition
- Add content for development, integration, staging, and
  production environment sections

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add ToC
- Complete "What is PostgreSQL" with definition and official docs link
- Add PostgreSQL server section
- Add PostgreSQL server in this project section

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Move deployment environment section before environment variable
- Add inspect subsections for env vars and PATH
- Add external docs links to each deployment environment type
- Fix anchor link from #deployment-environments to #deployment-environment
- Minor text clarifications (.env "dotenv file", backtick formatting)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- improve structure
- move distros part to a separate file
@deemp deemp merged commit 1bdeb57 into main Mar 3, 2026
1 of 2 checks passed
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