Pr/323#726
Closed
cristiZ1505 wants to merge 2 commits intocoleam00:mainfrom
Closed
Conversation
- Modified validate_supabase_url() to allow HTTP for local development - HTTP is now allowed for localhost, 127.0.0.1, host.docker.internal, and 0.0.0.0 - HTTPS is still required for production/non-local environments - Fixes server startup failure when using local Supabase with Docker
Collaborator
|
thank you for this contribution @cristiZ1505 feel free to reopen this and fill the PR template |
coleam00
added a commit
that referenced
this pull request
Apr 7, 2026
…phan conversations (#726) * fix(web): add cancel confirmation, surface rename errors, clean up orphan conversations (#708) Workflow cancel button now shows a confirmation dialog before cancelling. Conversation rename errors are displayed inline instead of silently logged. Orphan conversations created before a workflow run failure are cleaned up. Changes: - WorkflowProgressCard: wrap Cancel button with AlertDialog confirmation - ConversationItem: add renameError state and inline error display - WorkflowInvoker: track conversationId and delete on runWorkflow failure - WorkflowList: same orphan cleanup pattern as WorkflowInvoker Fixes #708 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * chore: Auto-commit workflow artifacts (archon-fix-github-issue) * fix(web): fix delete/rename ID mismatch, harden error handling (#708) - Fix broken delete/rename: was sending database UUID instead of platform_conversation_id to API endpoints that expect platform IDs - Hoist AlertDialog outside workflow status conditional to prevent unmounting mid-interaction when SSE status updates arrive - Add console.warn for orphan conversation cleanup failures instead of silently swallowing errors (CLAUDE.md compliance) - Scope orphan cleanup to only fire before runWorkflow starts to prevent deleting active conversations on client network errors - Use err: unknown with instanceof guards on all catch blocks - Surface useQuery errors with error UI instead of misleading empty state - Re-open delete dialog on error so users see failure messages Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Tyone88
pushed a commit
to Tyone88/Archon
that referenced
this pull request
Apr 16, 2026
…phan conversations (coleam00#726) * fix(web): add cancel confirmation, surface rename errors, clean up orphan conversations (coleam00#708) Workflow cancel button now shows a confirmation dialog before cancelling. Conversation rename errors are displayed inline instead of silently logged. Orphan conversations created before a workflow run failure are cleaned up. Changes: - WorkflowProgressCard: wrap Cancel button with AlertDialog confirmation - ConversationItem: add renameError state and inline error display - WorkflowInvoker: track conversationId and delete on runWorkflow failure - WorkflowList: same orphan cleanup pattern as WorkflowInvoker Fixes coleam00#708 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * chore: Auto-commit workflow artifacts (archon-fix-github-issue) * fix(web): fix delete/rename ID mismatch, harden error handling (coleam00#708) - Fix broken delete/rename: was sending database UUID instead of platform_conversation_id to API endpoints that expect platform IDs - Hoist AlertDialog outside workflow status conditional to prevent unmounting mid-interaction when SSE status updates arrive - Add console.warn for orphan conversation cleanup failures instead of silently swallowing errors (CLAUDE.md compliance) - Scope orphan cleanup to only fire before runWorkflow starts to prevent deleting active conversations on client network errors - Use err: unknown with instanceof guards on all catch blocks - Surface useQuery errors with error UI instead of misleading empty state - Re-open delete dialog on error so users see failure messages Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
joaobmonteiro
pushed a commit
to joaobmonteiro/Archon
that referenced
this pull request
Apr 26, 2026
…phan conversations (coleam00#726) * fix(web): add cancel confirmation, surface rename errors, clean up orphan conversations (coleam00#708) Workflow cancel button now shows a confirmation dialog before cancelling. Conversation rename errors are displayed inline instead of silently logged. Orphan conversations created before a workflow run failure are cleaned up. Changes: - WorkflowProgressCard: wrap Cancel button with AlertDialog confirmation - ConversationItem: add renameError state and inline error display - WorkflowInvoker: track conversationId and delete on runWorkflow failure - WorkflowList: same orphan cleanup pattern as WorkflowInvoker Fixes coleam00#708 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * chore: Auto-commit workflow artifacts (archon-fix-github-issue) * fix(web): fix delete/rename ID mismatch, harden error handling (coleam00#708) - Fix broken delete/rename: was sending database UUID instead of platform_conversation_id to API endpoints that expect platform IDs - Hoist AlertDialog outside workflow status conditional to prevent unmounting mid-interaction when SSE status updates arrive - Add console.warn for orphan conversation cleanup failures instead of silently swallowing errors (CLAUDE.md compliance) - Scope orphan cleanup to only fire before runWorkflow starts to prevent deleting active conversations on client network errors - Use err: unknown with instanceof guards on all catch blocks - Surface useQuery errors with error UI instead of misleading empty state - Re-open delete dialog on error so users see failure messages Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
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
3 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.
Pull Request
Summary
Changes Made
Type of Change
Affected Services
Testing
Test Evidence
Checklist
Breaking Changes
Additional Notes