Move .gitignore creation from init to logs download

[Copilot]

The .github/aw/logs/.gitignore file was created eagerly during gh aw init, requiring users to run init before downloading logs. Now created lazily on every gh aw logs invocation.

Changes

• pkg/cli/init.go: Removed ensureLogsGitignore() calls from initializeBasicRepository() and InitRepository()
• pkg/cli/logs_orchestrator.go: Added ensureLogsGitignore() at start of DownloadWorkflowLogs() - errors logged but non-blocking
• pkg/cli/init_command.go: Removed .gitignore creation from help text
• Tests: Updated to reflect new behavior - init no longer creates the file

Implementation

func DownloadWorkflowLogs(...) error {
    // Ensure .github/aw/logs/.gitignore exists on every invocation
    if err := ensureLogsGitignore(); err != nil {
        // Log but don't fail - this is not critical for downloading logs
        logsOrchestratorLog.Printf("Failed to ensure logs .gitignore: %v", err)
    }
    // ... rest of function
}

The ensureLogsGitignore() function remains unchanged - idempotent check-and-create logic in pkg/cli/git.go.

Original prompt

Do not eagerly generate a .gitignore file in .github/aw/logs in the init command.

The logs download code should ensure this file exists on every invocation.

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[Copilot]

Pull request overview

This PR moves the generation of .github/aw/logs/.gitignore from the init command to the logs download command, ensuring the file is created on every logs download invocation rather than only during repository initialization.

Changes:

• Removed .gitignore creation from both initializeBasicRepository and InitRepository functions in init.go
• Added .gitignore creation at the beginning of DownloadWorkflowLogs in logs_orchestrator.go with graceful error handling
• Updated init command help text to remove mention of .gitignore creation

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
pkg/cli/logs_orchestrator.go	Added call to ensureLogsGitignore() at the start of DownloadWorkflowLogs with non-critical error handling
pkg/cli/init.go	Removed ensureLogsGitignore() calls from both initializeBasicRepository and InitRepository functions
pkg/cli/init_command.go	Updated help text to remove mention of .gitignore creation for logs directory
pkg/cli/init_test.go	Removed test assertions that checked for .gitignore file creation, added comments explaining the change
pkg/cli/init_command_test.go	Removed test assertion that checked for .gitignore file creation in init tests

---

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