Skip to content

Added CinemaOS Provider#12

Draft
nischaldoescode wants to merge 1 commit intocinepro-org:devfrom
nischaldoescode:dev
Draft

Added CinemaOS Provider#12
nischaldoescode wants to merge 1 commit intocinepro-org:devfrom
nischaldoescode:dev

Conversation

@nischaldoescode
Copy link
Contributor

Pull Request

Description

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • 🌟 New feature (non-breaking change which adds functionality)
  • 📀 New provider (adding a new streaming source provider)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📝 Documentation update
  • ♻️ Code refactoring (no functional changes)
  • 🎨 Style/formatting changes
  • ⚡ Performance improvement
  • 🔧 Configuration change

Related Issues

Closes #

Changes Made

Provider Information (if applicable)

Provider Details

  • Provider Name:
  • Provider ID:
  • Base URL:
  • Content Types Supported:
    • Movies
    • TV Shows

Provider Testing

Tested with the following TMDB IDs:

  • Movies:
    • TMDB ID: 550 (Fight Club)
    • TMDB ID: _ (Your test movie)
  • TV Shows:
    • TMDB ID: 1399/1/1 (Game of Thrones S01E01)
    • TMDB ID: _ (Your test episode)

Test Results:

Paste relevant API response or logs here

Checklist

General

  • My code follows the code style of this project
  • I have run npm run format to format my code
  • My code builds without errors (npm run build)
  • I have tested my changes locally
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings or errors
  • I have read the CONTRIBUTING.md guidelines

Provider Checklist (if adding/modifying a provider)

  • Provider extends BaseProvider from @omss/framework
  • Provider has a unique id and descriptive name
  • Provider declares correct capabilities (movies/tv support)
  • Implements getMovieSources() and/or getTVSources() correctly
  • All streaming URLs use this.createProxyUrl() for proxying
  • Includes proper error handling with diagnostics
  • Uses this.console.log() for logging (not console.log())
  • Includes appropriate headers via this.HEADERS
  • Returns standardized ProviderResult format
  • Provider file is in src/providers/ directory
  • Provider exports the class properly for auto-discovery
  • Tested with multiple TMDB IDs for movies and/or TV shows
  • Works correctly with the proxy system
  • Handles edge cases (missing content, API errors)

Documentation

  • I have updated the README.md if necessary
  • I have added/updated comments in my code
  • I have documented any new environment variables
  • I have added examples if introducing new patterns

Testing

How Has This Been Tested?

Test Configuration:

  • Node.js version:
  • Operating System:
  • Cache type used: (memory/redis)

Testing Steps:

Test Output

Paste test output here

Screenshots (if applicable)

Additional Context

Performance Impact

  • No performance impact
  • Performance improvement (describe below)
  • Potential performance concern (describe below)

Details:

Breaking Changes

Details:

Dependencies

Reviewer Notes


For Maintainers

Merge Checklist

  • Code review completed
  • All CI checks passing
  • No merge conflicts
  • Provider tested by reviewer (if applicable)
  • Documentation updated (if needed)
  • Version bump needed? (yes/no)

@An0n-01 An0n-01 self-requested a review March 1, 2026 20:54
@An0n-01 An0n-01 added the enhancement New feature or request label Mar 1, 2026
@An0n-01
Copy link
Member

An0n-01 commented Mar 1, 2026

looks nice. does it work?

and i'll check it out before merging, so it might take a while till the merge is through.... (work yk)

@An0n-01
Copy link
Member

An0n-01 commented Mar 1, 2026

in the meantime, could you edit #12 (comment) and fill out the information? thanks🙏

@nischaldoescode
Copy link
Contributor Author

Didn't test it just wrote and pushed i think you could merge it to dev as of now...

@nischaldoescode
Copy link
Contributor Author

in the meantime, could you edit #12 (comment) and fill out the information? thanks🙏

Wydm ?

@An0n-01
Copy link
Member

An0n-01 commented Mar 1, 2026

Didn't test it just wrote and pushed i think you could merge it to dev as of now...

I won't merge untested changes😭😭. Not even on dev

@GLlgGL
Copy link

GLlgGL commented Mar 2, 2026

most of the links returned are not playable...Only Hindi server is playable which has hindi language..

@nischaldoescode
Copy link
Contributor Author

most of the links returned are not playable...Only Hindi server is playable which has hindi language..

Yeah they are forbidden....

@GLlgGL
Copy link

GLlgGL commented Mar 3, 2026

most of the links returned are not playable...Only Hindi server is playable which has hindi language..

Yeah they are forbidden....

There is Ophim server and another one that play but why showing wrong movie...but on website the movie is ok showing correctly...

@GLlgGL
Copy link

GLlgGL commented Mar 3, 2026

most of the links returned are not playable...Only Hindi server is playable which has hindi language..

Yeah they are forbidden....

There is Ophim server and another one that play but why showing wrong movie...but on website the movie is ok showing correctly...

The code shows on every result the Siperman movie lol...Maybe a logic bug on the code you should check

@An0n-01
Copy link
Member

An0n-01 commented Mar 3, 2026

I'll convert this PR to a draft, since it is not ready yet.

@An0n-01 An0n-01 marked this pull request as draft March 3, 2026 22:27
@An0n-01 An0n-01 removed their request for review March 3, 2026 22:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants