team #220
Open
charankumar001 wants to merge 708 commits into
Open
Conversation
- route() now validates payload at runtime instead of casting unknown: rejects non-objects with 400; ping requires zen or hook_id (422); issues requires string action + repository.full_name + issue object before reaching dispatchAgent - shared secret no longer falls back to the demo placeholder when --serve runs; GH_WEBHOOK_SECRET is required and the process exits with a clear message otherwise. The demo path keeps its constant for local fixture runs - node adapter caps request body at 1 MiB by tracking received bytes on the data handler, destroying the request and rejecting if the client streams more; the outer error handler now translates the cap into a 413 response instead of a generic 500 - parsePort validates --port the same way as lesson 12
- README fences tagged text + bash for MD040 - bargeFrames anchor clamped to length-1; empty-call guarded - probeWs adds 3s timeout that cleans listeners + closes socket - barge-in branch resets per-turn state (silence, partial, tool phase, filler) - synthCall splits on whitespace + filters blanks so empty scripts skip frames
- node adapter caps request body at 1 MiB by tracking received bytes on the data handler, destroying the request and surfacing a 413 through the outer error handler when the cap is exceeded - submit handler now does an explicit typeof check on correct after JSON.parse instead of leaning on zod; non-boolean correct fields return 400 with detail "correct must be boolean" - topoOrder validates that every prereq exists in the items set and throws a descriptive error for dangling prereqs before Kahn runs; after the loop, if any node still has indeg > 0 it throws a cycle error listing the stuck ids - parsePort validates --port the same way as the other lessons
…ipt-group-d # Conflicts: # catalog.json
…' into feat/phase-19-typescript-group-a
…' into feat/phase-19-typescript-group-b
…t-group-b feat(phase-19): add TypeScript skeletons for capstones 04/06/08
…ipt-group-a # Conflicts: # catalog.json
…' into feat/phase-19-typescript-group-a # Conflicts: # catalog.json
Previous design auto-committed catalog.json back to every PR branch on push. Each PR ended up with its own catalog.json delta, conflicting with main as soon as another PR merged. New design: - catalog-sync runs only on push to main (self-heals after merge) - catalog-drift-advisory runs on every PR (warning only, no commit) - readme-counts-drift rebuilds locally first, so it works on PRs too PRs never modify catalog.json. Contributors don't see conflicts. Main always reflects filesystem one commit after merge.
…t-group-a feat(phase-19): add TypeScript skeletons for capstones 01/02/03
…n-only ci(curriculum): regen catalog.json only on main push, not on PRs
…ipt-group-d # Conflicts: # catalog.json
…/rohitg00/ai-engineering-from-scratch into feat/phase-19-typescript-group-d # Conflicts: # catalog.json
- README.md: add text language tag to layout fenced block (MD040) - jobs.ts: advance timelines in list() so /jobs reflects live status - index.ts: build proper Headers in node adapter (avoid string/string[] coercion)
- README.md: add text language tag to layout fenced block (MD040)
- README.md: add text language tag to layout fenced block (MD040) - index.ts: build proper Headers in node adapter (avoid string/string[] coercion)
- README.md: add text language tag to layout fenced block (MD040) - index.ts: build proper Headers in node adapter (avoid string/string[] coercion)
…t-group-d feat(phase-19): add TypeScript skeletons for capstones 12/13/16/17
…auto-fix ci(curriculum): also auto-fix README counts on main push; PR check advisory
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What this PR does
Kind of change
Checklist
LESSON_TEMPLATE.mdstructure[Name](phases/...)), not bare textdocs/en.mdclaimsPhase / lesson
Notes for reviewer