Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename to Edge Worker #2

Merged
merged 14 commits into from
Jan 18, 2025
Merged

Rename to Edge Worker #2

merged 14 commits into from
Jan 18, 2025

Conversation

jumski
Copy link
Contributor

@jumski jumski commented Jan 17, 2025

I started with the name Supaworker, but later found out about typov-software/supaworker-js.

jumski added 10 commits January 17, 2025 16:33
Rename core class and related imports across multiple files to reflect new naming convention.
Changes include:
- Renamed Supaworker class to EdgeWorker
- Updated type SupaworkerConfig to EdgeWorkerConfig
- Updated import statements in all edge function files
- Modified error message for single call constraint
Migrate all database objects and functions from supaworker to edge_worker schema, including:
- Renaming schema from supaworker to edge_worker
- Updating all table, view, and function references
- Modifying SQL queries in TypeScript to use new schema name

This change updates the naming convention for the worker-related database objects while
maintaining the existing functionality.
Update import paths and project configuration to use new _src directory for Supabase edge functions
Deleted two performance test-related bash scripts:
- worker-test: Script for generating and processing test messages
- worker-test-results: Script for querying job performance metrics
Migrate project configuration, imports, and references from supaworker to edge-worker
- Update README.md with new project name and test command
- Modify deno.json import paths
- Rename project configuration in project.json
- Update Supabase-related commands and paths
- Adjust module exports and import statements
Rationale:
- The change is a file rename, which is best categorized as a refactoring
- The scope is (sql) to indicate the specific area of change
- The description concisely captures the essence of the modification
- No additional body is needed since the change is straightforward
- The type is 'refactor' because this is a structural change that doesn't add new functionality
or fix a bug
Rename package and update related references:
- Renamed `@pgflow/supaworker` to `@pgflow/edge-worker`
- Updated README.md package description
- Updated import_map.json import path
- Migrated all files from `pkgs/supaworker` to `pkgs/edge-worker`
Remove entire supabase-plugin project files, including configuration, package.json, and source
files.
Update nx.json to remove plugin-specific configurations related to the deleted project.
…ation

Change NX_DEFAULT_PROJECT from supaworker to edge-worker
Change Deno enabled path from 'supaworker' to 'edge-worker' in VS Code settings
Copy link

nx-cloud bot commented Jan 17, 2025

View your CI Pipeline Execution ↗ for commit d811523.

Command Status Duration Result
nx affected -t lint test build ✅ Succeeded 1m 53s View ↗

☁️ Nx Cloud last updated this comment at 2025-01-17 16:59:21 UTC

Improve type safety and modernize import statements across edge worker tests and source files.
Changes include:
- Replace generic 'any' types with more specific 'unknown'
- Update import statements from 'jsr:' to '@std/' namespace
- Modify function signatures for better type precision
- Remove unnecessary parameters and simplify test helper functions
Update dependency lock file with simplified package structure and version bump.
Removed some redundant specifiers and adjusted dependency references.
Remove additional test dependencies from test script
…sertThrows

Refactored test cases for WorkerState to:
- Remove async/await from test methods
- Replace assertRejects with assertThrows
- Simplify test method signatures
- Maintain existing test coverage and logic
@jumski jumski merged commit 214f0c8 into main Jan 18, 2025
2 checks passed
@jumski jumski deleted the rename-to-edge-worker branch January 18, 2025 13:38
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.

1 participant