NotificationsYou must be signed in to change notification settings
Fork26
Star82

docs: list NIP titles#401

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

tcheeric merged 1 commit intodevelopfromcodex/add-nip-title-to-stopped-nips-list

Aug 21, 2025

Merged

docs: list NIP titles#401

tcheeric merged 1 commit intodevelopfromcodex/add-nip-title-to-stopped-nips-list

Aug 21, 2025

Conversation

Copy link

Owner

tcheeric commentedAug 21, 2025

Why now?

The supported NIP list only referenced numbers, making it hard to identify each proposal.
Related issue: #000

What changed?

Show each supported NIP's title alongside its number in the README.

BREAKING

Review focus

Accuracy of the NIP titles and links.

Checklist

Scope ≤ 300 lines (or split/stack)
Title isverb + object (e.g., “Refactor auth middleware to async”)
Description links the issue and answers “why now?”
BREAKING flagged if needed
Tests/docs updated (if relevant)

https://chatgpt.com/codex/tasks/task_b_68a728bc78a88331851aa6c901ed06e5

docs: list NIP titles

1c069d8

tcheeric added the codex label

Aug 21, 2025

— with

ChatGPT Codex Connector

tcheeric requested a review fromCopilot

August 21, 2025 14:16

CopilotAI reviewed

Aug 21, 2025

View reviewed changes

Copy link

Contributor

CopilotAI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enhances the README documentation by adding descriptive titles alongside NIP numbers in the supported NIPs list, making it easier for developers to understand what each proposal covers without needing to follow individual links.

Key Changes

Added descriptive titles for all 23 supported NIPs in the README
Maintained existing link structure while improving readability

README.md

		-[NIP-9](https://github.com/nostr-protocol/nips/blob/master/09.md) - Event Deletion Request
		-[NIP-12](https://github.com/nostr-protocol/nips/blob/master/12.md) - Generic Tag Queries
		-[NIP-14](https://github.com/nostr-protocol/nips/blob/master/14.md) - Subject tag in Text events
		-[NIP-15](https://github.com/nostr-protocol/nips/blob/master/15.md) - Nostr Marketplace

Copy link

CopilotAIAug 21, 2025

Choose a reason for hiding this comment

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

The title should be 'Nostr Marketplace (for peer-to-peer marketplaces)' to match the official NIP-15 title format which includes the descriptive parenthetical.

Suggested change

	-[NIP-15](https://github.com/nostr-protocol/nips/blob/master/15.md) - Nostr Marketplace
	-[NIP-15](https://github.com/nostr-protocol/nips/blob/master/15.md) - Nostr Marketplace (for peer-to-peer marketplaces)

Copilot uses AI. Check for mistakes.

README.md

		-[NIP-46](https://github.com/nostr-protocol/nips/blob/master/46.md) - Nostr Remote Signing
		-[NIP-57](https://github.com/nostr-protocol/nips/blob/master/57.md) - Lightning Zaps
		-[NIP-60](https://github.com/nostr-protocol/nips/blob/master/60.md) - Cashu Wallets
		-[NIP-61](https://github.com/nostr-protocol/nips/blob/master/61.md) - Nutzaps

Copy link

CopilotAIAug 21, 2025

Choose a reason for hiding this comment

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

The title should be 'Nut Zaps' (two words) to match the official NIP-61 title format.

Suggested change

	-[NIP-61](https://github.com/nostr-protocol/nips/blob/master/61.md) -Nutzaps
	-[NIP-61](https://github.com/nostr-protocol/nips/blob/master/61.md) -Nut Zaps