Skip to content

Conversation

@akn19 akn19 closed this Sep 12, 2025
@akn19 akn19 reopened this Sep 12, 2025
@akn19 akn19 closed this Sep 12, 2025
@akn19 akn19 reopened this Sep 12, 2025
Copy link
Contributor

@MikeZeDev MikeZeDev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

First : fix linting.

Copy link
Contributor

@MikeZeDev MikeZeDev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Must change some code
  • I am wondering why bothering to update that for hakuneko
  • if you have found errors in Haruneko support by all mean tell us. I see differences i cant explain.


canHandleURI(uri) {
return new RegExp(
`^${this.url.replace(/[-/\\^$*+?.()|[\]{}]/g, "\\$&")}/comic/[^/]+$`,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove canHandleURI entirely. We dont use such precise control in hakuneko. Plus that regexp is way too complex .

} = await this.fetchAPI(`./v/${chapter.id}`);

return images.map((img) => {
let url = typeof img === "string" ? img : img.url || img.source;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is that? is there something i missed in haruneko?
Can you provide sample url?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

plus if img is a string how can it have an "url" property?

return chapters.map(({ slug, number }) => {
let title = number.toString().trim();
if (/^\d+(\.\d+)?$/.test(title)) {
title = `Chapter ${title}`;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Prefixing with chapter will lead to inconsistencies with haruneko and. .. can you explain what you are doing with the title?

async getMangasFromPage(page) {
const { data } = await this.fetchAPI(`./contents?page=${page}`);
return data
? data.map(({ slug, title }) => ({
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not sure that Hakuneko stable 6.1.7 is ok with the ? : syntax . Perhaps i am just too careful.

super.label = "WestManga";
this.tags = ["manga", "manhua", "manhwa", "indonesian"];
this.url = "https://westmanga.me";
this.queryMangaTitle = "h1";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why using a class property for that ?

}

async _getPages(chapter) {
const {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

inline this


async _getMangaFromURI(uri) {
const slug = uri.pathname.split("/").pop();
const {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

inline this

@akn19 akn19 closed this Sep 16, 2025
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