Skip to content

first pass on week 2 actual assignments#37

Open
paris-fractal wants to merge 2 commits intomainfrom
week-2-impl
Open

first pass on week 2 actual assignments#37
paris-fractal wants to merge 2 commits intomainfrom
week-2-impl

Conversation

@paris-fractal
Copy link
Collaborator

@paris-fractal paris-fractal commented Jun 7, 2025

Important

Reorganizes week 2 chatbot curriculum by adding assignments.md and removing outdated files, detailing a structured daily plan for building a chatbot with specific technologies.

  • New Curriculum Structure:
    • Adds assignments.md for week 2 chatbot project, detailing daily tasks and goals.
    • Monday: Set up Next.js app with t3.gg, Vercel AI, and shadcn components.
    • Tuesday: Implement tool usage, switch to Supabase/PostgreSQL, and use tRPC.
    • Wednesday: Add user authentication with NextAuth and per-user chat storage.
    • Thursday: Plan to support full-text search.
  • Removals:
    • Deletes 1-chatbot.md and rough-plan.md, consolidating content into assignments.md.

This description was created by Ellipsis for 4a8bfdc. You can customize this summary. It will automatically update as commits are pushed.

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

Caution

Changes requested ❌

Reviewed everything up to 04d1976 in 1 minute and 26 seconds. Click for details.
  • Reviewed 199 lines of code in 3 files
  • Skipped 0 files when reviewing.
  • Skipped posting 4 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. curriculum/weeks/2-chatbot/assignments.md:30
  • Draft comment:
    Typo: 'Inspriation' should be 'Inspiration'.
  • Reason this comment was not posted:
    Decided after close inspection that this draft comment was likely wrong and/or not actionable: usefulness confidence = 10% vs. threshold = 50% While this is technically correct, typo fixes in documentation are very minor issues. The meaning is still clear despite the typo. According to the rules, we should not make comments that are obvious or unimportant. This feels like a very minor issue that doesn't materially impact the usefulness of the documentation. The typo could be confusing for non-native English speakers. Also, maintaining high quality documentation is important for educational materials. While documentation quality matters, this typo is still easily understood and fixing it isn't worth the overhead of a PR comment and review cycle. This comment should be deleted as it points out a very minor issue that doesn't materially impact the documentation's usefulness.
2. curriculum/weeks/2-chatbot/assignments.md:4
3. curriculum/weeks/2-chatbot/assignments.md:30
  • Draft comment:
    Typo: 'Inspriation' should be spelled 'Inspiration'.
  • Reason this comment was not posted:
    Decided after close inspection that this draft comment was likely wrong and/or not actionable: usefulness confidence = 10% vs. threshold = 50% While the spelling error is real, this is a curriculum document where typos should be fixed. However, the rules state not to make comments that are obvious or unimportant. Spelling errors are typically caught by basic spell checkers and IDE tools. This isn't a code logic issue that requires review attention. The spelling error could affect students reading the document. Documentation quality is important for educational materials. While documentation quality matters, this is a simple typo that can be caught by basic tools. PR comments should focus on more substantial issues that require human review. Delete this comment as it points out a trivial spelling error that doesn't warrant a PR comment.
4. curriculum/weeks/2-chatbot/assignments.md:35
  • Draft comment:
    Typo: 'neccessary' should be spelled 'necessary'.
  • Reason this comment was not posted:
    Marked as duplicate.

Workflow ID: wflow_SyYu9LznDqti0wBy

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

# Pre-work

- Grokking Simplicity Ch 3
- [Next.js App Router Tutorial](https://nextjs.org/learn/dashboard-app))
Copy link
Contributor

Choose a reason for hiding this comment

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

Extra ')' in URL link; remove the additional closing parenthesis.

Suggested change
- [Next.js App Router Tutorial](https://nextjs.org/learn/dashboard-app))
- [Next.js App Router Tutorial](https://nextjs.org/learn/dashboard-app)

- Streams responses from the AI
- has unique chats (e.g. I can open a new chat, get redirected to `/chat/abc-123`, and go back to that URL later and continue the work)
- (this can happen via local files for now)
- Has nice styling using `chadcn`
Copy link
Contributor

Choose a reason for hiding this comment

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

Inconsistent naming: chadcn should likely be shadcn to match the earlier reference.

Suggested change
- Has nice styling using `chadcn`
- Has nice styling using `shadcn`

# Tuesday
Your app:
- has at least one tool the AI can use
- uses tRPC to communicate with the server (when neccessary)
Copy link
Contributor

Choose a reason for hiding this comment

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

Typo: 'neccessary' should be 'necessary'.

Suggested change
- uses tRPC to communicate with the server (when neccessary)
- uses tRPC to communicate with the server (when necessary)

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

Important

Looks good to me! 👍

Reviewed 4a8bfdc in 1 minute and 45 seconds. Click for details.
  • Reviewed 124 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 6 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. curriculum/weeks/2-chatbot/assignments.md:1
  • Draft comment:
    Updated main header to include week context; ensure consistency with overall curriculum hierarchy.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None
2. curriculum/weeks/2-chatbot/assignments.md:6
  • Draft comment:
    Removed extra closing parenthesis from the Next.js App Router link.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None
3. curriculum/weeks/2-chatbot/assignments.md:17
  • Draft comment:
    Corrected the UI library name from 'chadcn' to 'shadcn' for clarity.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None
4. curriculum/weeks/2-chatbot/assignments.md:41
  • Draft comment:
    Fixed typo: corrected 'neccessary' to 'necessary'.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None
5. curriculum/weeks/2-chatbot/assignments.md:36
  • Draft comment:
    Updated day headers to use H2 instead of H1 for improved structural consistency.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None
6. curriculum/weeks/2-chatbot/assignments.md:34
  • Draft comment:
    Typo: 'Inspriation' should be corrected to 'Inspiration'.
  • Reason this comment was not posted:
    Comment was on unchanged code.

Workflow ID: wflow_Ch8qkd87fudjXtQn

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

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