Skip to content

Add next five GitHub domain todo specs and remaining-domain estimate#42

Merged
pmcelhaney merged 1 commit into
mainfrom
copilot/create-todos-for-next-5-domains
Jun 9, 2026
Merged

Add next five GitHub domain todo specs and remaining-domain estimate#42
pmcelhaney merged 1 commit into
mainfrom
copilot/create-todos-for-next-5-domains

Conversation

Copilot AI commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds five new .github/todo proposals for the next GitHub simulator domains and computes the updated remaining-domain count. This scopes upcoming implementation work and makes backlog size explicit (31 domains remaining).

Original Prompt

Create todos for building out the next 5 GitHub domains. Create a pull request. Then estimate how many domains are remaining.

Manual acceptance tests

  • .github/todo includes emojis.md, licenses.md, markdown.md, meta.md, and rate-limit.md.
  • Each new todo contains YAML front matter with title plus ## Summary, ## Scope, and ## Implementation plan.
  • Each todo is scoped to its matching route domain and references context/scenario/tests work.
  • Remaining-domain estimate in the PR is consistent with route-domain groups still using $.response[...].random() (36 total, 5 newly queued, 31 remaining).

Tasks

  • Domain backlog expansion
    • Added five new task specs under .github/todo for: Emojis, Licenses, Markdown, Meta, and Rate Limit.
  • Task structure standardization
    • Used the existing proposal format (front matter + summary/scope/implementation plan) for each domain file.
  • Remaining work estimation
    • Counted route-domain groups still returning randomized responses and subtracted this batch to produce the remaining estimate.
  • Example snippet
    ---
    title: Implement Emojis domain in the GitHub API simulator
    ---
    
    ## Summary
    ...
    ## Scope
    ...
    ## Implementation plan
    ...

Copilot AI self-assigned this Jun 9, 2026
Copilot AI review requested due to automatic review settings June 9, 2026 17:16
Copilot AI review requested due to automatic review settings June 9, 2026 17:16
Copilot AI changed the title Add todo tasks for next 5 GitHub domains Add next five GitHub domain todo specs and remaining-domain estimate Jun 9, 2026
Copilot AI requested a review from pmcelhaney June 9, 2026 17:16
@pmcelhaney pmcelhaney marked this pull request as ready for review June 9, 2026 17:19
Copilot AI review requested due to automatic review settings June 9, 2026 17:19
@pmcelhaney pmcelhaney added this pull request to the merge queue Jun 9, 2026
Merged via the queue into main with commit f59f4c4 Jun 9, 2026
6 of 8 checks passed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds five new .github/todo task specs to queue up upcoming GitHub simulator domain implementations (Emojis, Licenses, Markdown, Meta, Rate Limit) and document the next work items using the repo’s existing todo format.

Changes:

  • Added todo specs for five GitHub API domains under .github/todo/.
  • Standardized each todo with front matter and ## Summary / ## Scope / ## Implementation plan.
  • Documented planned context/scenario/test work for each domain.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
.github/todo/emojis.md New todo spec for implementing deterministic GET /emojis.
.github/todo/licenses.md New todo spec for Licenses domain (needs scope/plan to include /licenses/{license} as well).
.github/todo/markdown.md New todo spec for Markdown domain (needs scope/plan to include /markdown/raw as well).
.github/todo/meta.md New todo spec for implementing deterministic GET /meta.
.github/todo/rate-limit.md New todo spec for implementing deterministic GET /rate_limit.

Comment thread .github/todo/licenses.md

### Route files to implement

- `routes/licenses.ts` — `GET /licenses`
Comment thread .github/todo/licenses.md
Comment on lines +19 to +21
3. Update `routes/licenses.ts` to use context-backed list logic.
4. Seed representative license fixtures in scenarios and include them in `seedGitHub`.
5. Add context unit tests in `test/licenses.context.test.ts` and HTTP-level route tests.
Comment thread .github/todo/markdown.md

### Route files to implement

- `routes/markdown.ts` — markdown rendering operations
Comment thread .github/todo/markdown.md
Comment on lines +19 to +21
3. Update `routes/markdown.ts` to delegate behavior to context methods.
4. Add startup fixtures or helper scenario setup as needed for deterministic rendering tests.
5. Add context unit tests and HTTP-level route tests for success and invalid-input behavior.
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.

3 participants