starred/ALwrity

Fork

You've already forked ALwrity

mirror of https://github.com/AJaySi/ALwrity.git synced 2026-04-25 17:05:56 +03:00

Code Issues 82 Projects Releases 4 Packages Wiki Activity

[PR #404] [MERGED] fix: resolve onboarding session not found warnings and frontend build… #711

New issue

Closed

opened 2026-03-13 21:05:04 +03:00 by kerem · 0 comments

kerem commented

2026-03-13 21:05:04 +03:00

Owner

Copy link

📋 Pull Request Information

Original PR: https://github.com/AJaySi/ALwrity/pull/404
Author: @AJaySi
Created: 3/9/2026
Status: ✅ Merged
Merged: 3/9/2026
Merged by: @AJaySi

Base: main ← Head: feature/website-maker-phase1

📝 Commits (1)

b3cc83e fix: resolve onboarding session not found warnings and frontend build OOM

📊 Changes

13 files changed (+1708 additions, -20 deletions)

View changed files

📝 backend/api/onboarding_utils/endpoints_config_data.py (+88 -0)
📝 backend/api/onboarding_utils/endpoints_core.py (+2 -2)
📝 backend/api/onboarding_utils/onboarding_control_service.py (+2 -2)
📝 backend/api/onboarding_utils/step_management_service.py (+4 -4)
➕ backend/api/onboarding_utils/website_automation_service.py (+250 -0)
➕ backend/models/user_website_request.py (+121 -0)
📝 backend/services/onboarding/api_key_manager.py (+12 -9)
➕ backend/services/onboarding/website_automation_service.py (+206 -0)
➕ backend/services/onboarding/website_intake_service.py (+285 -0)
➕ backend/services/onboarding/website_style_service.py (+439 -0)
➕ backend/services/user_website_service.py (+295 -0)
📝 frontend/package.json (+2 -1)
📝 frontend/src/stores/workflowStore.ts (+2 -2)

📄 Description

… OOM

Use canonical Clerk user id (clerk_user_id) across all onboarding entrypoints to ensure consistent OnboardingSession.user_id lookup
Fix API key persistence in api_key_manager.py to use correct APIKey model columns (session_id, provider, key)
Increase Node heap for frontend build to 8GB and add build:nomap script to disable sourcemaps and reduce memory usage
Update onboarding endpoints (endpoints_core.py, onboarding_control_service.py, step_management_service.py) to prefer clerk_user_id over id
Fix frontend workflowStore.ts TypeScript error by returning WorkflowError instance
Add website_automation_service.py for onboarding automation

Pull Request

📝 Description

Brief description of changes made in this PR.

🔄 Type of Change

🐛 Bug fix (non-breaking change which fixes an issue)
✨ New feature (non-breaking change which adds functionality)
💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
📚 Documentation update
🎨 Style/UI changes
♻️ Code refactoring
⚡ Performance improvements
🧪 Test additions/updates

🎯 Related Issues

Closes #(issue number)
Fixes #(issue number)
Related to #(issue number)

🧪 Testing

Backend tests pass
Frontend tests pass
Manual testing completed
Cross-browser testing (if applicable)
Mobile testing (if applicable)

📸 Screenshots (if applicable)

Add screenshots to help explain your changes.

Before

After

🏷️ Component/Feature

Which component or feature is affected?

Blog Writer
SEO Dashboard
Content Planning
Facebook Writer
LinkedIn Writer
Onboarding
Authentication
API
Database
GSC Integration
Subscription System
Monitoring/Billing
Documentation
Other: _______________

📋 Checklist

My code follows the project's style guidelines
I have performed a self-review of my own code
I have commented my code, particularly in hard-to-understand areas
I have made corresponding changes to the documentation
My changes generate no new warnings
I have added tests that prove my fix is effective or that my feature works
New and existing unit tests pass locally with my changes
Any dependent changes have been merged and published

ALwrity-Specific Checklist

API endpoints follow RESTful conventions
AI service integrations handle rate limits and errors gracefully
Content generation includes proper validation and sanitization
Database migrations are included if schema changes are made
Environment variables are documented in env_template.txt
Security considerations have been addressed
Performance impact has been considered
User experience is consistent with existing features

🔍 Code Quality

Code is properly formatted
No console.log statements left in production code
Error handling is implemented where needed
Performance considerations have been addressed
Security considerations have been addressed

📚 Documentation

README updated (if needed)
API documentation updated (if needed)
Code comments added for complex logic
Changelog updated (if applicable)

🚀 Deployment Notes

Any special deployment considerations or environment variables needed.

🔗 Additional Context

Add any other context about the pull request here.

👥 Reviewers

Tag specific reviewers if needed:
@AJaySi @uniqueumesh @DikshaDisciplines

Thank you for contributing to ALwrity! 🎉

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/AJaySi/ALwrity/pull/404 **Author:** [@AJaySi](https://github.com/AJaySi) **Created:** 3/9/2026 **Status:** ✅ Merged **Merged:** 3/9/2026 **Merged by:** [@AJaySi](https://github.com/AJaySi) **Base:** `main` ← **Head:** `feature/website-maker-phase1` --- ### 📝 Commits (1) - [`b3cc83e`](https://github.com/AJaySi/ALwrity/commit/b3cc83ed6e2a88fac491be7d4b74922dc3a07526) fix: resolve onboarding session not found warnings and frontend build OOM ### 📊 Changes **13 files changed** (+1708 additions, -20 deletions) <details> <summary>View changed files</summary> 📝 `backend/api/onboarding_utils/endpoints_config_data.py` (+88 -0) 📝 `backend/api/onboarding_utils/endpoints_core.py` (+2 -2) 📝 `backend/api/onboarding_utils/onboarding_control_service.py` (+2 -2) 📝 `backend/api/onboarding_utils/step_management_service.py` (+4 -4) ➕ `backend/api/onboarding_utils/website_automation_service.py` (+250 -0) ➕ `backend/models/user_website_request.py` (+121 -0) 📝 `backend/services/onboarding/api_key_manager.py` (+12 -9) ➕ `backend/services/onboarding/website_automation_service.py` (+206 -0) ➕ `backend/services/onboarding/website_intake_service.py` (+285 -0) ➕ `backend/services/onboarding/website_style_service.py` (+439 -0) ➕ `backend/services/user_website_service.py` (+295 -0) 📝 `frontend/package.json` (+2 -1) 📝 `frontend/src/stores/workflowStore.ts` (+2 -2) </details> ### 📄 Description … OOM - Use canonical Clerk user id (clerk_user_id) across all onboarding entrypoints to ensure consistent OnboardingSession.user_id lookup - Fix API key persistence in api_key_manager.py to use correct APIKey model columns (session_id, provider, key) - Increase Node heap for frontend build to 8GB and add build:nomap script to disable sourcemaps and reduce memory usage - Update onboarding endpoints (endpoints_core.py, onboarding_control_service.py, step_management_service.py) to prefer clerk_user_id over id - Fix frontend workflowStore.ts TypeScript error by returning WorkflowError instance - Add website_automation_service.py for onboarding automation # Pull Request ## 📝 Description Brief description of changes made in this PR. ## 🔄 Type of Change - [ ] 🐛 Bug fix (non-breaking change which fixes an issue) - [ ] ✨ New feature (non-breaking change which adds functionality) - [ ] 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected) - [ ] 📚 Documentation update - [ ] 🎨 Style/UI changes - [ ] ♻️ Code refactoring - [ ] ⚡ Performance improvements - [ ] 🧪 Test additions/updates ## 🎯 Related Issues Closes #(issue number) Fixes #(issue number) Related to #(issue number) ## 🧪 Testing - [ ] Backend tests pass - [ ] Frontend tests pass - [ ] Manual testing completed - [ ] Cross-browser testing (if applicable) - [ ] Mobile testing (if applicable) ## 📸 Screenshots (if applicable) Add screenshots to help explain your changes. ### Before  ### After  ## 🏷️ Component/Feature Which component or feature is affected? - [ ] Blog Writer - [ ] SEO Dashboard - [ ] Content Planning - [ ] Facebook Writer - [ ] LinkedIn Writer - [ ] Onboarding - [ ] Authentication - [ ] API - [ ] Database - [ ] GSC Integration - [ ] Subscription System - [ ] Monitoring/Billing - [ ] Documentation - [ ] Other: _______________ ## 📋 Checklist - [ ] My code follows the project's style guidelines - [ ] I have performed a self-review of my own code - [ ] I have commented my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the documentation - [ ] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] Any dependent changes have been merged and published ### ALwrity-Specific Checklist - [ ] API endpoints follow RESTful conventions - [ ] AI service integrations handle rate limits and errors gracefully - [ ] Content generation includes proper validation and sanitization - [ ] Database migrations are included if schema changes are made - [ ] Environment variables are documented in env_template.txt - [ ] Security considerations have been addressed - [ ] Performance impact has been considered - [ ] User experience is consistent with existing features ## 🔍 Code Quality - [ ] Code is properly formatted - [ ] No console.log statements left in production code - [ ] Error handling is implemented where needed - [ ] Performance considerations have been addressed - [ ] Security considerations have been addressed ## 📚 Documentation - [ ] README updated (if needed) - [ ] API documentation updated (if needed) - [ ] Code comments added for complex logic - [ ] Changelog updated (if applicable) ## 🚀 Deployment Notes Any special deployment considerations or environment variables needed. ## 🔗 Additional Context Add any other context about the pull request here. ## 👥 Reviewers Tag specific reviewers if needed: @AJaySi @uniqueumesh @DikshaDisciplines --- **Thank you for contributing to ALwrity!** 🎉 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-03-13 21:05:04 +03:00

closed this issue
added the
pull-request
label

No Branch/Tag specified

main

codex/task-title-3y5pbt

codex/task-title-lsrpho

codex/task-title

codex/locate-and-render-brollinfopanel-component

codex/import-scene-composition-symbols-in-broll_service

codex/refactor-preview-generation-flow

codex/standardize-chart-preview-route-usage

codex/import-b-roll-handler-and-update-router

codex/implement-podcast-research-response-updates

codex/add-cost-estimation-feature-to-podcastdashboard

codex/remove-heuristic-calculators-and-update-estimates-bhvhnh

codex/remove-heuristic-calculators-and-update-estimates

codex/implement-dedicated-podcastcostest-model

codex/add-podcast-maker-feature-documentation

codex/review-flat-file-context-system-implementation

codex/create-podcast-maker-best-practices-documentation

codex/add-podcast-maker-journey-pages-for-personas

codex/refactor-podcast-maker-documentation-structure

dependabot/npm_and_yarn/frontend/npm_and_yarn-7d3393088f

codex/log-skipped-and-loaded-items

codex/create-startup-phases-for-podcast-demo-tugrzv

codex/create-startup-phases-for-podcast-demo

codex/add-api-feature-profile-status-endpoint

codex/refactor-routermanager-to-use-registry

codex/add-new-modules-for-feature-management

codex/downgrade-api-key-warning-logging

codex/refactor-onboarding-loading-logic

codex/add-ci-lint-check-for-user_id-patterns

codex/remove-sessionlocal-import-and-add-health-check

codex/refactor-dubbed_audio_dir-to-tenant-path-resolver

codex/update-task-metadata-and-api-responses

codex/verify-podcast-analysis-and-cost-tracking

codex/assert-api-endpoints-on-startup

codex/add-env-controlled-guard-for-stripe-key

codex/remove-feature-flag-for-production-demo

codex/add-mode-aware-redirect-logic-in-pricingpage

codex/document-deployment-configuration-for-demo-mode

codex/ensure-api.subscription.router-always-included

codex/guard-onboarding-manager-and-endpoints

codex/wrap-router-inclusion-in-demo-mode-check

codex/add-podcast-only-demo-mode-toggle

dependabot/npm_and_yarn/frontend/npm_and_yarn-f5ea90b336

codex/update-reset_current_billing_period-logic

codex/update-error-handling-in-monitoring-middleware

codex/move-stripe_plan_price_mapping-to-env-driven-config

codex/refactor-billingservice-to-use-authenticated-user

codex/update-user-access-verification-for-routes

session/agent_eae6711f-e04b-4d48-8d66-43ee051feb7b

v0.5.0

v0.3

v0.2

v0.1

Labels

Clear labels

RAG knowledgebase Memory

Mirrored from GitHub Pull Request

No labels

RAG knowledgebase Memory

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/ALwrity#711

Reference in a new issue

Repository

starred/ALwrity

Title

Body

No description provided.

Delete branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?

Rows
Columns