Skip to content

refractor w/ pixi,click & my_favs downldr#46

Open
corednoir wants to merge 8 commits intoeverdrone:masterfrom
corednoir:pixi_click
Open

refractor w/ pixi,click & my_favs downldr#46
corednoir wants to merge 8 commits intoeverdrone:masterfrom
corednoir:pixi_click

Conversation

@corednoir
Copy link
Copy Markdown

@corednoir corednoir commented Jan 21, 2026

just got around this a few days ago, and while adapting into another project, took the time to improve what it felt right.

Happily to work from here into a safe ground.

to run one only need to

pixi run nts --help

edit:
just noticed #44 , the logic to grab the link checks for a valid one prior to call yt_dlp, i found one broken link during one session and was able to avoid it

@corednoir
Copy link
Copy Markdown
Author

with last comit it resolves #44

just confirm some eps with 404 @ mixcloud

------------------------------
404 Not Found: https://www.mixcloud.com/NTSRadio/soup-to-nuts-w-shy-one-100-dangelo/, attempt 1/5
mixcloud link 404
mixcloud_api failed https://api.mixcloud.com/search/?q=Soup+To+Nuts+w+Shy+One+100%+D'Angelo+15th+October+2025&type=cloudcast
Soup To Nuts w- Shy One - 100% D'Angelo - 2025-10-15 -- https://soundcloud.com/user-202286394-991268468/soup-to-nuts-w-shy-one-151025

@everdrone
Copy link
Copy Markdown
Owner

Thank you for the PR!
What's the motivation behind pixi over uv? I was considering uv a while ago already to ease up the development process a bit.

@corednoir
Copy link
Copy Markdown
Author

As pixi uses uv under the hood, i have opt for it instead, since the scope is bigger and not restricted to py.
Have got no problems other than some in setting global tools.

@everdrone
Copy link
Copy Markdown
Owner

Hi, sorry for the delay in answering.

I just ran the pixi run nts --help command and found a few issues.

  • magic (libmagic) is a new requirement that needs to be installed by the user
  • PATH_CDN is asserted and will error out if not defined
    Both of these prevent to even run --help.

The most common problem this repository faces is beginners trying to just download music and ending up having to face python errors or installation issues for the first time, this seems a step in the wrong direction.

I appreciate the effort and the refactor is much needed, but the tool should be installed effortlessly by newcomers (no external dependencies).
This is the first time I'm using pixie so apologies if I'm missing something.

@corednoir
Copy link
Copy Markdown
Author

i totally agree with that, thats why i started to use pixi in the first place
you didnt miss anything

add a fallback to less deep check of file type, that's what magic is being used

chck if anything is still missing :)

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.

2 participants