fix(aluno): Portal do Aluno login, dev tooling, and error log#58
Merged
Conversation
…edirect
Middleware was blocking /aluno/login for unauthenticated users, causing
a silent redirect loop back to /login when clicking 'Acessar Portal do
Aluno'. Also fixes auth action redirecting to /aluno (no page) instead
of /aluno/dashboard after successful aluno login.
- middleware.ts: exclude /aluno/login from isAlunoRoute protected check
- auth.ts: redirect('/aluno') -> redirect('/aluno/dashboard')
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Two files created: - docs/dev-errors.md: team-visible log with full context per error - .claude/errors.md: compact AI context mirror 10 errors from current session logged (ERR-001 to ERR-010), covering middleware bugs, test mocking patterns, commitlint rules, branch naming, and PR template requirements. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Add to .gitignore: - .remember/ (session handoff notes, per-developer ephemeral state) - .specify/feature.json (active feature pointer, changes per session) Track previously unversioned files: - CLAUDE.md — project AI context and coding guidelines - .claude/skills/ — project-specific speckit skills (14 skills) - .specify/ — speckit tooling: templates, scripts, extensions, integrations, constitution - specs/001-fix-type-safety-bugs/ — feature spec documentation Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (64)
📝 WalkthroughWalkthroughThis pull request introduces a comprehensive Spec Kit project framework including thirteen workflow skill definitions, a complete Git branching extension with Bash/PowerShell scripts, shared helper utilities, configuration schemas, and markdown templates for specification, planning, and task management. Minor application changes add role-based access control and correct authentication redirect paths. Changes
Sequence Diagram(s)Sequence diagram generation is not applicable to this change set. The modifications primarily introduce documentation, configuration, templates, and utility scripts that define declarative workflows and structural organization rather than introducing new sequential interactions between 3+ distinct components with conditional control flows. Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Poem
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
This was referenced Apr 10, 2026
Merged
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
1 participant
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.
Description
Three independent improvements shipped together after the US00 merge:
Type of Change
feat— New featurefix— Bug fixrefactor— Code refactor (no behavior change)test— Tests onlydocs— Documentation onlychore— Build, config, toolingci— CI/CD pipelineperf— Performance improvementRelated Documents
docs/dev-errors.mdChecklist
npm run typecheck— zero errorsnpm run lint— zero errorsnpm run format:check— no formatting issuesnpm run test:coverage— all tests passdocs/dev-errors.md,CHANGELOG.mdCommits
7caffdafix(aluno)— Middleware excluding/aluno/loginfrom protected routes;redirect('/aluno')→redirect('/aluno/dashboard')eb68a5adocs—docs/dev-errors.md+.claude/errors.mdcom 10 erros da sessãobde8d06chore—.gitignoreadds.remember/e.specify/feature.json; tracksCLAUDE.md,.claude/skills/,.specify/,specs/001-*Test Plan
/login→ click "Acessar Portal do Aluno" → lands on/aluno/loginana.silva@example.com/123456→ lands on/aluno/dashboard🤖 Generated with Claude Code
Summary by CodeRabbit
New Features
Bug Fixes
Documentation