-
Notifications
You must be signed in to change notification settings - Fork 15
chore: update Deno requirement to 2.1.x and add context7 config #407
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
chore: update Deno requirement to 2.1.x and add context7 config #407
Conversation
|
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
|
View your CI Pipeline Execution ↗ for commit 4501f43
☁️ Nx Cloud last updated this comment at |
4fe8656 to
9c83cab
Compare
scripts/supabase-start.sh
Outdated
| if nc -z localhost "$port" 2>/dev/null; then | ||
| # Find process holding the port using ss | ||
| local pid=$(ss -lpn 2>/dev/null | grep ":$port " | grep -oE "pid=[0-9]+" | cut -d= -f2 | head -1) | ||
| if [ -n "$pid" ]; then | ||
| echo -e "${YELLOW}Killing process holding port $port (PID: $pid)${NC}" | ||
| kill -9 "$pid" 2>/dev/null || true | ||
| fi | ||
| fi | ||
| done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The ss command is Linux-specific and not available on macOS. This function will fail silently on macOS systems, leaving ports occupied and causing "port already in use" errors when starting Supabase.
Fix by adding cross-platform support:
cleanup_port_processes() {
local project_dir="$1"
local ports=$(get_project_ports "$project_dir")
for port in $ports; do
if nc -z localhost "$port" 2>/dev/null; then
# Cross-platform: use lsof on macOS, ss on Linux
local pid=""
if command -v lsof >/dev/null 2>&1; then
pid=$(lsof -ti:"$port" 2>/dev/null | head -1)
elif command -v ss >/dev/null 2>&1; then
pid=$(ss -lpn 2>/dev/null | grep ":$port " | grep -oE "pid=[0-9]+" | cut -d= -f2 | head -1)
fi
if [ -n "$pid" ]; then
echo -e "${YELLOW}Killing process holding port $port (PID: $pid)${NC}"
kill -9 "$pid" 2>/dev/null || true
fi
fi
done
sleep 2
}| if nc -z localhost "$port" 2>/dev/null; then | |
| # Find process holding the port using ss | |
| local pid=$(ss -lpn 2>/dev/null | grep ":$port " | grep -oE "pid=[0-9]+" | cut -d= -f2 | head -1) | |
| if [ -n "$pid" ]; then | |
| echo -e "${YELLOW}Killing process holding port $port (PID: $pid)${NC}" | |
| kill -9 "$pid" 2>/dev/null || true | |
| fi | |
| fi | |
| done | |
| if nc -z localhost "$port" 2>/dev/null; then | |
| # Cross-platform: use lsof on macOS, ss on Linux | |
| local pid="" | |
| if command -v lsof >/dev/null 2>&1; then | |
| pid=$(lsof -ti:"$port" 2>/dev/null | head -1) | |
| elif command -v ss >/dev/null 2>&1; then | |
| pid=$(ss -lpn 2>/dev/null | grep ":$port " | grep -oE "pid=[0-9]+" | cut -d= -f2 | head -1) | |
| fi | |
| if [ -n "$pid" ]; then | |
| echo -e "${YELLOW}Killing process holding port $port (PID: $pid)${NC}" | |
| kill -9 "$pid" 2>/dev/null || true | |
| fi | |
| fi | |
| done | |
| sleep 2 |
Spotted by Graphite Agent
Is this helpful? React 👍 or 👎 to let us know.
e79e896 to
37edaf1
Compare
37edaf1 to
91487f9
Compare
91487f9 to
f67ad3f
Compare
…bility - Added new context7.json schema with flow rules and best practices - Updated flow compilation instructions to specify Deno v2.1.x or higher - Corrected Deno version references in flow and installation docs - Modified tutorial and index pages to reflect new Deno runtime version - Updated Deno version in AI web scraper tutorial for consistency
f67ad3f to
4501f43
Compare
Merge activity
|
# Setup context7.json and update missed Deno version specifiers This PR adds `context7.json` configuration file with project rules and documentation guidelines Additionally: - Updated the documentation to specify Deno 2.1.x or higher as the required version for pgflow compilation, replacing the previous requirement of 1.45.2 or higher. The change is reflected across multiple documentation files. - Modified the lefthook.yml script to automatically accept symlinks by adding the `--yes` flag to the Claude symlink script These changes ensure consistent documentation about version requirements and improve the developer experience with automated tooling.
🔍 Preview Deployment: Website✅ Deployment successful! 🔗 Preview URL: https://pr-407.pgflow.pages.dev 📝 Details:
_Last updated: _ |

Setup context7.json and update missed Deno version specifiers
This PR adds
context7.jsonconfiguration file with project rules and documentation guidelinesAdditionally:
--yesflag to the Claude symlink scriptThese changes ensure consistent documentation about version requirements and improve the developer experience with automated tooling.