This reverts commit 402e72743d9a9bd6dbdc7c24f1f20d62cededf11.

Replace custom token acquisition logic with SDK's built-in TokenSource
function as suggested by @pietern. This simplifies the implementation
by delegating token lifecycle management to the SDK configuration.

Changes:
- Use cfg.GetTokenSource() instead of auth.AcquireToken()
- Remove custom token_loader abstraction (libs/auth/token_loader.go)
- Revert cmd/auth/token.go to original implementation
- Remove tokenAcquireTimeout constant (handled by SDK)

This addresses review feedback from @pietern on Nov 17, 2025.
Reference: https://gist.github.com/pietern/41d02acc2907416244789c7408fb232f

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

Relocate logstream from libs/logstream to libs/apps/logstream to make
it clearer that this package is specific to Databricks Apps functionality.

Changes:
- Move libs/logstream/streamer.go to libs/apps/logstream/streamer.go
- Move libs/logstream/streamer_test.go to libs/apps/logstream/streamer_test.go
- Update import in cmd/workspace/apps/logs.go

This addresses review feedback from @pietern on Nov 17, 2025.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

Group individual const declarations into a single const block for better
code organization and readability.

Changes:
- Convert six individual const declarations to const block
- Apply alignment formatting

This addresses review feedback from @pkosiec on Nov 14, 2025.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

When using --tail-lines with -f (follow), the buffer was flushing too
early, showing all historical logs instead of just the last N lines.

The tail buffer maintains a rolling window of the last N lines, but it
was flushing as soon as it reached N lines when following, rather than
waiting for the prefetch window to collect all historical logs first.

Fix: Always respect the flush deadline (prefetch window) regardless of
follow mode. This allows historical logs to stream in and the rolling
buffer to maintain only the truly last N lines before flushing.

Example with --tail-lines 10 -f:
- Before: Shows lines 1-10, then 11-1000+ (all logs)
- After: Shows lines 991-1000, then new logs (correct)

This addresses review feedback from @pkosiec on Nov 14, 2025.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

Replace magic number (30 seconds) with defaultHandshakeTimeout constant
for better code maintainability.

Changes:
- Add defaultHandshakeTimeout = 30 * time.Second to const block
- Use constant in newLogStreamDialer instead of inline value

This addresses review feedback from @pkosiec on Nov 12/14, 2025.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

When following app logs (--follow) and the app stops or is deleted,
the logs command now detects this and exits gracefully instead of
retrying indefinitely.

Implementation:
- Added AppStatusChecker callback to logstream.Config
- Check app status before each reconnect attempt in follow mode
- Also check when connection closes normally during follow
- Exit with clear error message when app is stopped/deleting/error state

This addresses @pietern's comment about the command not stopping when
the app is stopped during tail.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>