Skip to content

Update setup_fabric.sh to install dependencies if not present#9

Merged
gdellis merged 4 commits into
mainfrom
feature/code-improvements
Feb 10, 2026
Merged

Update setup_fabric.sh to install dependencies if not present#9
gdellis merged 4 commits into
mainfrom
feature/code-improvements

Conversation

@gdellis
Copy link
Copy Markdown
Owner

@gdellis gdellis commented Feb 10, 2026

Summary

This PR updates the setup_fabric.sh script to automatically install missing dependencies before proceeding with the Fabric installation. Previously, the script would only check for dependencies and exit with an error if any were missing. Now it leverages the ensure_command function from dependencies.sh to install any missing dependencies.

Changes Made

  1. Modified the check_dependencies function to use ensure_command for each dependency
  2. Added proper package name mappings for op, ffmpeg, yt-dlp, and curl
  3. Set auto_install to true to automatically install missing dependencies
  4. Improved error handling to report which dependencies failed to install

Benefits

  • Improves user experience by reducing manual steps
  • Makes the Fabric installation process more robust
  • Ensures all required dependencies are present before installation

Glenn Ellis added 4 commits February 10, 2026 10:54
… 1Password gracefully

- Updated check_dependencies function to actually install missing dependencies using ensure_command
- Enhanced create_config function to handle 1Password dependency more gracefully:
  * Check if 1Password CLI is installed and user is signed in
  * Gracefully fall back when 1Password is not configured
  * Provide helpful logging messages to guide users on enabling YouTube integration
  * Make YouTube API key optional in config file
- Dependencies now properly installed: op (1password-cli), ffmpeg, yt-dlp, curl
…ut 1Password

- Added interactive prompt for users to manually enter YouTube API key
- Improved messaging and instructions for enabling YouTube integration
- Allow users to skip YouTube integration entirely if they prefer
- Provide clear information on how to enable YouTube integration later
- Maintains backward compatibility with existing 1Password integration
@gdellis gdellis merged commit e09c52b into main Feb 10, 2026
1 check passed
@gdellis gdellis deleted the feature/code-improvements branch February 10, 2026 18:36
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