Skip to content

Add SSH remote session support with fixed port - #5

Merged
backnotprop merged 2 commits into
backnotprop:mainfrom
dizzyd:ssh-remote-support
Dec 31, 2025
Merged

backnotprop merged 2 commits into
backnotprop:mainfrom
dizzyd:ssh-remote-support

Conversation

@dizzyd

@dizzyd dizzyd commented Dec 31, 2025

Copy link
Copy Markdown
Contributor

I use Claude Code over a SSH connection to a cross-compiling server. Being able to access Plannotator over the network is extremely useful. This branch adds SSH remote session support to Plannotator.

  • Auto-detect SSH sessions via SSH_TTY/SSH_CONNECTION env vars
  • Use fixed port 19432 over SSH for predictable port forwarding
  • Add PLANNOTATOR_PORT env var for custom port configuration
  • Add retry loop (5 attempts) for port conflicts
  • Print SSH config hint when running remotely
  • Skip browser auto-open in SSH sessions

- Auto-detect SSH sessions via SSH_TTY/SSH_CONNECTION env vars
- Use fixed port 19432 over SSH for predictable port forwarding
- Add PLANNOTATOR_PORT env var for custom port configuration
- Add retry loop (5 attempts) for port conflicts
- Print SSH config hint when running remotely
- Skip browser auto-open in SSH sessions
@backnotprop

Copy link
Copy Markdown
Owner

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

🤖 Generated with Claude Code

- If this code review was useful, please react with 👍. Otherwise, react with 👎.

@backnotprop
backnotprop merged commit 7ec2937 into backnotprop:main Dec 31, 2025
@backnotprop

Copy link
Copy Markdown
Owner

merged, thanks!

@backnotprop

Copy link
Copy Markdown
Owner

backnotprop added a commit that referenced this pull request Jan 7, 2026
* Add SSH remote session support with fixed port

- Auto-detect SSH sessions via SSH_TTY/SSH_CONNECTION env vars
- Use fixed port 19432 over SSH for predictable port forwarding
- Add PLANNOTATOR_PORT env var for custom port configuration
- Add retry loop (5 attempts) for port conflicts
- Print SSH config hint when running remotely
- Skip browser auto-open in SSH sessions

* integration test for ssh

---------

Co-authored-by: Michael Ramos <mdramos8@gmail.com>
backnotprop added a commit that referenced this pull request May 2, 2026
Combines #5's clean header (mascot, badges, links) with #1's narrative
hook and alternating image/text feature showcase. All five features get
consistent zigzag layout. Install instructions moved out to linked docs.

For provenance purposes, this commit was AI assisted.
@backnotprop

Copy link
Copy Markdown
Owner

@dizzyd

We're building Plannotator Workspaces, where agent plans, specs, and reviews can be collaborated on and distilled into standards and rules. Your annotations and reviews compound into a golden ruleset planning and review agents can use, and CI/CD can verify against.

Joining the waitlist gets you lifetime free access as an early contributor. https://plannotator.ai/workspaces/. Check the OSS contributor box.

The OSS core isn't changing: same free and open source plugin, always.

Thanks for #5.

backnotprop added a commit that referenced this pull request Jul 1, 2026
- #1 Copy feedback now copies the full feedback (code + editor + PR description
  + PR comment notes) and gates on totalAnnotationCount, matching Send Feedback.
  Moved below the feedbackMarkdown/count memos to avoid a TDZ.
- #3 PR description/comment notes (not line-anchored) are seeded into the GitHub
  review body when opening the platform dialog, quoted — so they post in the
  summary and a prose-only review still has something to submit.
- #4 Drafts now persist + restore descriptionAnnotations + commentAnnotations
  (useCodeAnnotationDraft payload/restore extended; App wires save + restore).
- #5 Clicking a "PR comment" note in the sidebar opens/focuses the PR Overview
  panel and selects + scrolls to the source comment (token-based signal through
  ReviewStateContext, consumed by PRCommentsTab).

Left per discussion: #2 (stale notes on PR switch), #6 (live-cache timer).
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.

2 participants