Skip to content

Add getdnote.com feature#177

Merged
sebst merged 3 commits intomainfrom
copilot/add-getdnote-com-feature
Mar 23, 2026
Merged

Add getdnote.com feature#177
sebst merged 3 commits intomainfrom
copilot/add-getdnote-com-feature

Conversation

Copy link
Contributor

Copilot AI commented Mar 23, 2026

Adds a devcontainer feature for dnote — a single-binary, SQLite-backed command-line notebook.

Changes

  • src/getdnote.com/ — new feature using the gh release install method
  • src/getdnote.com/install.sh — customises the standard github_list_releases / github_get_tag_for_version helpers to handle dnote's non-standard cli-v{version} release tag prefix (assets follow the pattern dnote_{version}_linux_{arch}.tar.gz)
  • test/getdnote.com/test.sh — verifies installation via dnote version
  • README.md — adds entry alphabetically between fzf and github.com-cli

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • www.getdnote.com
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>Add getdnote.com feature</issue_title>
<issue_description></issue_description>

Comments on the Issue (you are @copilot in this section)


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Copilot AI changed the title [WIP] Add getdnote.com feature Add getdnote.com feature Mar 23, 2026
Copilot AI requested a review from sebst March 23, 2026 11:15
@sebst sebst marked this pull request as ready for review March 23, 2026 11:58
Copy link
Member

@sebst sebst left a comment

Choose a reason for hiding this comment

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

tests fail

Copy link
Contributor Author

Copilot AI commented Mar 23, 2026

tests fail

Fixed in f036beb. The dnote release archive stores the binary as ./dnote (with a leading ./), but the script was looking for dnote (without it), causing tar: dnote: Not found in archive. Updated binaryPathInArchive to ./dnote, which also makes the stripComponents calculation yield 1 (strips the ./ prefix), so the binary is correctly extracted to /usr/local/bin/dnote.

Copilot AI requested a review from sebst March 23, 2026 12:04
@sebst sebst merged commit c62d99e into main Mar 23, 2026
7 checks passed
@sebst sebst deleted the copilot/add-getdnote-com-feature branch March 23, 2026 12:12
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.

Add getdnote.com feature

2 participants