zsh: remove sl='ssh lightsail' alias

[idvorkin]

Removes the sl='ssh lightsail' alias from shared/zsh_include.sh so sl is free for Sapling source control.

asl and slnf lightsail aliases are kept.

Summary by CodeRabbit

• Chores
  • Updated shell aliases configuration: removed one alias and introduced two new aliases for remote connection workflows.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 89063f0f-e545-4380-b03c-d9b1a89108ac

📥 Commits

Reviewing files that changed from the base of the PR and between b61e9ab and 0eecdc1.

📒 Files selected for processing (1)

• shared/zsh_include.sh

💤 Files with no reviewable changes (1)

• shared/zsh_include.sh

---

📝 Walkthrough

Walkthrough

The shell configuration updates SSH connection aliases in shared/zsh_include.sh. The existing sl alias is removed and replaced with two new aliases: asl for autossh-based connections with keepalive monitoring, and slnf for direct SSH connections without port forwarding.

Changes

SSH Connection Aliases

Layer / File(s)	Summary
SSH alias updates shared/zsh_include.sh	The sl alias (ssh lightsail) is removed. Two new aliases are added: asl for autossh with port 20000 keepalive to lightsail_no_forward, and slnf for direct SSH to lightsail_no_forward.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

A rabbit hops through shell configs with glee,
Old sl hops away, now we have two to see,
With asl and slnf, the connections run true,
SSH aliases dancing—the shell's work to do! 🐰🔌

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately reflects the main change in the PR—removing the sl alias from zsh configuration.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch remove-sl-alias

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Copilot]

Pull request overview

Removes the sl='ssh lightsail' zsh alias from shared/zsh_include.sh to free up sl for Sapling usage, while retaining the other Lightsail-related aliases.

Changes:

• Deleted alias sl='ssh lightsail' from the shared zsh include file.
• Kept existing Lightsail aliases (asl, slnf) unchanged.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.