Skip to content

v1.4.0 #33

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 28 commits into from
Jul 4, 2025
Merged

v1.4.0 #33

merged 28 commits into from
Jul 4, 2025

Conversation

rtuszik
Copy link
Owner

@rtuszik rtuszik commented Jul 1, 2025 •
edited
Loading

Modularize Starwarden into a proper Python package, refactor monolithic script into reusable modules, add Apprise notifications, and update documentation for version 1.4.0.

New Features:

  • Add Apprise-based notifications with APPRISE_URLS support. (partially addresses Uptime Kuma integration #23)
  • Introduce a modular package structure (main, config, github_api, linkwarden_api, tui, utils) replacing the single starwarden.py script.

Enhancements:

  • Centralize logging with RichHandler and RotatingFileHandler in a shared logger utility.
  • Simplify entrypoint to delegate to new starwarden.main module and set a DOCKERIZED flag in Dockerfile.
  • Handle existing link exception in linkwarden 409 response. (closes Response status code: 409 - Link already exists #34)

Build:

  • Add apprise==1.9.3 to requirements.txt and bump requests and python-dotenv versions.
  • Expose DOCKERIZED environment variable in Dockerfile.

Documentation:

  • Update README to include APPRISE_URLS environment variable.
  • Adjust installation and usage instructions to reflect the new package structure.

rtuszik added 20 commits July 1, 2025 13:58
@rtuszik
create starwarden module
40fc479
@rtuszik
use @rtuszik default logger snippet
7a96905
@rtuszik
refactor: make starwarden.py serve as entrypoint
966c282
@rtuszik
add notification module
cd7b03c
@rtuszik
create config module
61f5d3f
@rtuszik
feat: move core logic to main.py
0de8236
@rtuszik
feat: create modules for linkwarden and github api logic
e8dea1f
@rtuszik
feat: create module for rich terminal interface
b563bd0
@rtuszik
fix: correct logger import
360ba49
@rtuszik
fix: call logger
22b3b81
@rtuszik
fix(github_api): remove redundant starting index for page number iter…
050c3bf 
…ation
@rtuszik
refactor(github_api): simplify starred repository retrieval using `yi…
6545f03 
…eld from`
@rtuszik
use f-string instead of string concatenation
840de6b
@rtuszik
fix(logger): update default backup count for log files to 2
59db7f2
@rtuszik
fix: notify module import and unused logger setting
5e795f0
@rtuszik
fix: update requests package version and fix logger import path
e589f8b
@rtuszik
chore: update python-dotenv dep
7f0b36d
@rtuszik
fix(config): Change collection ID type from string to integer
cfdce3b
@rtuszik
add notification on app launch
9d43f26
@rtuszik
feat: add DOCKERIZED environment variable to control apprise notifica…
5f5e919 
…tions
@rtuszik rtuszik added this to the 2.0.0 milestone Jul 1, 2025
@socket-security Socket Security
Copy link

socket-security bot commented Jul 1, 2025 •
edited
Loading

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security		 Vulnerability Quality Maintenance License
Addedapprise@​1.9.374100100100100
Updatedrequests@​2.32.3 ⏵ 2.32.499 +1100100100100
Updatedpython-dotenv@​1.1.0 ⏵ 1.1.199100100100100

View full report

@rtuszik rtuszik changed the title v2.0.0 v1.4.0 Jul 1, 2025
rtuszik added 5 commits July 1, 2025 22:09
@rtuszik
refactor: update docker-compose.yml and CLI argument for collection ID
c77fc40
@rtuszik
docs(README.md): add documentation for APPRISE_URLS environment variable
02c4130 
docs(README.md): update table formatting
@rtuszik
docs(README): improve formatting of code blocks and tables for better…
7ad60fd 
… readability
@rtuszik
refactor(logger): replace sys.stdout with RichHandler for improved co…
3e949e2 
…nsole logging

fix(logger): change default console log level to WARNING
fix(logger): change default file log level to INFO
@rtuszik
fix(linkwarden_api): handle 409 conflict for existing links in Linkwa…
b758960 
…rden API

refactor(linkwarden_api): change error response content to .text for better readability
fix(main): skip processing for globally existing links in Linkwarden
fix(logger): set default enable_console_logging to true
refactor(logger): use RichHandler directly for console logging
@rtuszik rtuszik self-assigned this Jul 4, 2025
@rtuszik rtuszik removed this from the 2.0.0 milestone Jul 4, 2025
@rtuszik

This comment was marked as resolved.

Sign in to view
sourcery-ai[bot]

This comment was marked as resolved.

Sign in to view

rtuszik added 3 commits July 4, 2025 13:54
@rtuszik
feat(main.py): send notification with upload statistics after run update
d76dba6 
refactor(main.py): remove initial "Starwarden is now active" notification
refactor(logger.py): load environment variables directly in logger utility
@rtuszik
refactor(linkwarden_api): replace magic string with constant for link…
db5d434 
… existence

fix(linkwarden_api): correct cursor logic for fetching existing links
refactor(main): use constant for link existence check
refactor(config): remove redundant print statement for missing environment variables
refactor(logger): improve logging handler management to prevent duplicates
fix(logger): add fallback error message for log file handler creation
@rtuszik
docs(README.md): remove redundant table rows
73076d6 
feat(docker-compose.yml): add volume for starwarden.log to persist logs
sourcery-ai[bot]

This comment was marked as resolved.

Sign in to view

@rtuszik rtuszik marked this pull request as ready for review July 4, 2025 12:17
@rtuszik rtuszik merged commit 54a4fdb into main Jul 4, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sourcery-ai sourcery-ai[bot] sourcery-ai[bot] approved these changes

Assignees

@rtuszik rtuszik

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Response status code: 409 - Link already exists
1 participant
@rtuszik