Skip to content

Conversation

@kohai-io
Copy link
Contributor

Summary

Recent Kobo firmware versions have replaced Pocket with Instapaper. This PR updates NickelMenu to support the new Instapaper functionality.

Changes

  • src/action_cc.cc: Replace library:pocket with library:instapaper and update symbol to _ZN15LibraryNavMixin21showInstapaperLibraryEv
  • res/doc: Update documentation to reflect Instapaper instead of Articles/Pocket

Testing

  • Tested on Kobo Libra Colour kobo13 v4.43.23418 firmware
  • Symbol verified using strings command on libnickel.so.1.0.0
  • Menu item successfully opens Instapaper library view

Usage

menu_item:main:Instapaper:nickel_open:library:instapaper

- Replace library:pocket with library:instapaper in nickel_open action
- Update symbol to _ZN15LibraryNavMixin21showInstapaperLibraryEv
- Update documentation to reflect Instapaper instead of Articles/Pocket
- Tested and working on Kobo Libra Colour kobo13 v4.43.23418 firmware
@kohai-io kohai-io requested a review from pgaskin as a code owner August 27, 2025 17:20
Copy link
Owner

@pgaskin pgaskin left a comment

Choose a reason for hiding this comment

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

This seems to be at least partly AI generated. There's no such thing as "latest" for the symbol tests.

On a more constructive note, leave pocket in and just add instapaper as an addition option.

I haven't tested it, but the symbol looks correct.

@kohai-io
Copy link
Contributor Author

Hi Patrick @pgaskin. Apologies I'm no C/C++ developer so yes used Sonnet 4 to help me with this change. I've tried to dig through libnickel.so.1.0.0 in the 4.43.23418 firmware but I can't identify a specific version for libnickel itself, so I've updated the comment to 4.43.23418 * (assuming that means that version onwards) in action_cc.cc - I hope that's correct, if not do let me know and also if it's good practice to update the symbol testing tool too.

@terriblephrases
Copy link

Thank you, this worked on my Kobo Libra 2 on version 4.38.23429.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants