Skip to content

Enhanced SIP Template with Improved Structure and Fixed YAML#8

Open
gregsantos wants to merge 2 commits intomainfrom
gregsantos-template-update
Open

Enhanced SIP Template with Improved Structure and Fixed YAML#8
gregsantos wants to merge 2 commits intomainfrom
gregsantos-template-update

Conversation

@gregsantos
Copy link
Collaborator

Summary

This PR enhances the Story Improvement Proposal (SIP) template to provide better guidance for proposal authors while maintaining usability. The changes include structural improvements, new required sections for comprehensive proposals, optional sections for flexibility, and fixes to the YAML front matter.

Key Changes

🔧 Fixed YAML Front Matter

  • Fixed parsing errors in the metadata section that prevented proper document processing
  • Improved structure with proper YAML formatting and data types
  • Added new metadata fields for better proposal tracking and organization

📋 Enhanced Required Sections

  • Abstract: New concise technical summary section
  • Specification: More detailed technical design requirements with RFC 2119 compliance
  • Security Considerations: Mandatory security analysis (critical for decentralized networks)
  • Performance Implications: Structured performance analysis framework
  • Implementation Plan: Phased development and deployment strategy
  • Community Consensus: Formal tracking of stakeholder feedback and governance

🔄 New Optional Sections

  • New Terminology (optional): For proposals introducing new concepts
  • Reference Implementation (optional): Pseudocode and implementation sketches
  • Economic Impact (optional if not applicable): Token economics and incentive analysis
  • Migration Guide (optional - required only for breaking changes): Step-by-step transition plans
  • Related Work (optional): Prior art and future work considerations
  • Appendices (optional): Additional reference materials

🏗️ Improved Structure

  • Better organized impact analysis with separate user, developer, network, and economic considerations
  • Enhanced compatibility section covering backward, forward, and ecosystem integration
  • Comprehensive dependency tracking for technical and process dependencies
  • Structured alternatives analysis with clear pros/cons evaluation

Benefits

  • Reduces proposal quality issues by providing clear guidance on required content
  • Improves consistency across all SIPs with standardized sections
  • Maintains flexibility with optional sections for different proposal types
  • Enhances governance process with better community consensus tracking
  • Fixes technical issues that prevented proper document parsing

Note: The enhanced template draws inspiration from successful improvement proposal processes in other decentralized networks while being tailored specifically for Story Protocol. New proposals should use the enhanced template for better consistency and completeness.

Add New Required and Optional Sections with details and examples. Fix YAML heading.
@gregsantos gregsantos requested review from Ramarti and ramtinms June 27, 2025 19:01
sponsors: First Sponsor's Name (expert@example.org)
created: (fill me in with today's date, YYYY-MM-DD)
type: Standard/Ecosystem/Meta
created: 2025-06-27
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why fill the date if this is not an SIP, but an improvement of the template?

created: (fill me in with today's date, YYYY-MM-DD)
type: Standard/Ecosystem/Meta
created: 2025-06-27
type: Standard
Copy link
Collaborator

Choose a reason for hiding this comment

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

If this is still a template, it should have the 3 options.
If you want this proposal to be a Meta IP, then copy the template and modify that file, not the template directly

- What new APIs or tools are needed?
- How will this impact existing development workflows?

### Network Impact
Copy link
Collaborator

Choose a reason for hiding this comment

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

This one is also optional if not applicable?


## Performance Implications (optional)

### Computational Impact
Copy link
Collaborator

Choose a reason for hiding this comment

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

How is this different from Performance Implication

2. Phase 2: Integration and testing
3. Phase 3: Deployment and migration

### Testing Strategy
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think implementation details, dev phases, testing and deployment are OOO for a standard definition.
Ideally a standard may have different implementations with different strategies.


## Documentation and Examples

### Tutorial and Usage Examples
Copy link
Collaborator

Choose a reason for hiding this comment

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

This also feel more like implementation

- What is their level of support or concern?
- How were conflicts resolved?

### Governance Considerations
Copy link
Collaborator

Choose a reason for hiding this comment

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

Governance process should be independent of individual SIPs, unless this is an SIP targeting governance.
Ideally there is a common governance process to settle which SIPs are included in a fork. For app/ecosystem standards it may not need voting for example.

@ramtinms
Copy link
Collaborator

ramtinms commented Sep 2, 2025

Thanks for the proposal and already making the updates to the templates, do you mind also create a markdown file for these changes (beside the PR description), so we can keep track of changes in the history as well.

## Abstract

Provide a concise technical summary of the proposal in 2-3 sentences. This should be understandable to someone familiar with the Story Protocol ecosystem but not necessarily the specific problem domain.

Copy link
Collaborator

Choose a reason for hiding this comment

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

we have limited length size that is broken here now.

@@ -1,85 +1,220 @@
---
number: 'XXXXX' (update this part after PR creation)
number: 'XXXXX'
Copy link
Collaborator

Choose a reason for hiding this comment

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

maybe we still keep the details on when to update this number

- Second Author's name (author2@example.org)
sponsors: First Sponsor's Name (expert@example.org)
created: (fill me in with today's date, YYYY-MM-DD)
created: 'XXXXX'
Copy link
Collaborator

Choose a reason for hiding this comment

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

same here

Comment on lines -8 to +13
status: Draft
supersedes: (optional - fill this in if the SIP supersedes a previous SIP)
superseded-by: (optional - fill this in if the SIP is superseded by a
subsequent SIP)
extends: (optional - fill this in if the SIP extends the design of a
previous SIP)
status: Draft/Accepted/Implemented/Released/Stagnant/Withdrawn
supersedes: null
superseded-by: null
extends: null
Copy link
Collaborator

Choose a reason for hiding this comment

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

same here, please don't remove the template helpers here

concise—details should be provided in the below sections.
Provide a brief summary of this proposal and why it's needed. Keep it concise—details should be provided in the below sections.

Please include links to relevant community discussion threads.
Copy link
Collaborator

Choose a reason for hiding this comment

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

same here

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.

3 participants