Skip to content

Bruno - #1

Open
davipt wants to merge 1 commit into
mainfrom
bruno
Open

Bruno#1
davipt wants to merge 1 commit into
mainfrom
bruno

Conversation

@davipt

@davipt davipt commented Mar 3, 2026

Copy link
Copy Markdown
Owner

Summary

Brief description of what this PR does and why.

Changes

  • Ignore .DS_Store
  • Fix port number to not be formatted for local locale (e.g., "host:12 345")
  • log content-type and content-length only on POST
  • only show stop button if running
  • added auto-start toggle and functionality
  • fix showSettingsWindow deprecation and use SettingsLink
  • made default system prompt a reusable constant
  • enabled dead-code-stripping as recommended by xcode

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Refactor / cleanup
  • Other (describe):

Testing

Describe how you tested your changes:

  • Server starts and the menu bar icon appears
  • POST /v1/chat/completions returns a valid response
  • Built-in chat UI works correctly
  • No regressions in existing behavior

Checklist

  • I've read the Contributing Guide
  • My code follows the existing style (Swift 6.0, no third-party dependencies)
  • I've updated documentation if behavior changed
  • No debug prints or commented-out code left behind

Related Issues

Closes #

…e (e.g., "host:12 345"); log content-type and content-length only on POST; only show stop button if running; added auto-start toggle and functionality; fix showSettingsWindow deprecation and use SettingsLink; made default system prompt a reusable constant; enabled dead-code-stripping as recommended by xcode
pull Bot pushed a commit that referenced this pull request Apr 2, 2026
Replaces the bearer token authentication with a three-layer security
model that requires no user configuration:

1. Loopback-only binding — server unreachable from the network
2. CORS origin allowlist — only localhost and perspectiveintelligence.app
3. Host header validation — prevents DNS rebinding attacks

This fixes #2 (Continue extension 401 errors) and addresses #1 with
proper CORS protection instead of tokens. Local apps connect freely
since they already have full machine access.

Adds perspectiveintelligence.app to the CORS allowlist for browser-
direct streaming (Basic tier web users).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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