Skip to content

docs: Sprint 2.5 - Namespace Architecture Documentation Updates#44

Merged
pipewrk merged 5 commits intomainfrom
43-sprint-2.5-documentation-updates
Oct 3, 2025
Merged

docs: Sprint 2.5 - Namespace Architecture Documentation Updates#44
pipewrk merged 5 commits intomainfrom
43-sprint-2.5-documentation-updates

Conversation

@pipewrk
Copy link
Contributor

@pipewrk pipewrk commented Oct 3, 2025

Summary

This PR contains the documentation updates for Sprint 2.5 (Namespace Architecture Fix). These changes prepare the documentation for the upcoming namespace auto-detection implementation.

Changes Made

Public Documentation Updates

  • docs/guide/events.md:

    • Updated event naming convention examples to show namespace auto-detection
    • Added examples of plugin slug detection (e.g., acme-blog.* instead of wpk.*)
    • Demonstrated explicit namespace override patterns
    • Clarified framework vs application event branding
  • docs/guide/resources.md:

    • Added namespace configuration examples in defineResource
    • Showed auto-detected namespace usage
    • Demonstrated explicit namespace override functionality
    • Updated event and store key examples

Documentation Philosophy

These changes maintain WP Kernel's zero-ceremony approach while showing users they can:

  1. 90% case: Just use defineResource - namespace auto-detected
  2. 9% case: Override namespace when needed
  3. 1% case: Framework uses explicit wpk namespace

Related Work

  • Issue: Sprint 2.5 - Documentation Updates for Namespace Architecture #43 (Sprint 2.5 Documentation Updates)
  • Sprint Documentation: Created in information/sprints/ (not tracked in git)
    • Sprint 2.5 main document with goals, scope, technical approach
    • Task breakdown with 6 tasks including security vulnerability fix
    • Updated internal docs (Product Specification, Event Taxonomy Reference)

Next Steps

After this PR merges:

  1. Implement namespace auto-detection system
  2. Fix security vulnerability in E2E utils (new Function() usage)
  3. Update resource definitions to support namespace parameter
  4. Update E2E utils for namespace awareness
  5. Audit codebase for hardcoded namespace assumptions

Impact

  • Business Identity: Enables plugins to use their own brand in events
  • Framework Positioning: Clear separation of framework vs application concerns
  • Developer Experience: Preserves zero-ceremony while adding flexibility
  • Security: Documents plan to fix code injection vulnerability

- Update events.md to show namespace auto-detection examples
- Show override patterns for explicit namespace control
- Update resources.md with namespace configuration examples
- Demonstrate plugin slug auto-detection (e.g., acme-blog.*)
- Clarify framework vs application event branding

Prepares documentation for Sprint 2.5 namespace architecture fix.

Related: #43
Copilot AI review requested due to automatic review settings October 3, 2025 01:48
Copy link
Contributor

Copilot AI 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 updates the documentation for Sprint 2.5, introducing namespace architecture changes that enable automatic namespace detection based on plugin context. The changes prepare the documentation for upcoming implementation while maintaining WP Kernel's zero-ceremony philosophy.

Key changes:

  • Added namespace auto-detection examples showing plugin-specific event naming (e.g., acme-blog.* instead of wpk.*)
  • Documented explicit namespace override patterns for advanced use cases
  • Clarified separation between framework events (wpk.*) and application events (plugin-specific)

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
docs/guide/resources.md Added namespace auto-detection examples and explicit override patterns for defineResource
docs/guide/events.md Updated event naming examples to show namespace auto-detection and framework vs application event separation
Comments suppressed due to low confidence (1)

docs/guide/resources.md:1

  • Inconsistent event naming examples. These lines still show 'wpk.thing.' namespace, but should reflect the new namespace auto-detection pattern established earlier in the document (e.g., 'acme-blog.thing.').
# Resources

addAction('wpk.cache.invalidated', 'my-plugin/debug', (payload) => {
console.log('Cache invalidated:', payload.keys);
});

Copy link

Copilot AI Oct 3, 2025

Choose a reason for hiding this comment

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

Inconsistent markdown code block delimiters. Line 148 uses four backticks (`````) while line 160 uses four backticks where three should be used (```).

Copilot uses AI. Check for mistakes.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed in commit d1ff8a9, please check

pipewrk and others added 4 commits October 3, 2025 09:53
Co-authored-by: Copilot <[email protected]>
Signed-off-by: Jason Joseph Nathan <[email protected]>
…tion

- Update main README with namespace auto-detection patterns
- Fix hardcoded wpk.* references in package READMEs
- Update API docs and guides with namespace examples
- Clarify framework vs user event namespacing
- Complete Sprint 2.5 Task 1 documentation requirements
…nts guide

- Remove incorrect opening markdown delimiter at file start
- Fix inconsistent 4-backtick usage throughout the file
- Ensure all code blocks use standard 3-backtick delimiters
- Remove wrapping markdown block around final section
- All 8 code blocks now properly paired with matching delimiters

Resolves markdown parsing issues in docs/guide/events.md
@pipewrk pipewrk merged commit d4c2774 into main Oct 3, 2025
7 checks passed
pipewrk added a commit that referenced this pull request Oct 3, 2025
- Creates 7 issues for remaining Sprint 2.5 tasks (Tasks 2-8)
- Task 1 (Documentation) already completed via PR #44
- Covers namespace architecture implementation (Tasks 2-6)
- Includes Sprint 2 completion work (Tasks 7-8)
- Ready to create milestone and issues for 3-day sprint

Usage: ./scripts/create-sprint-2.5-issues.sh
@pipewrk pipewrk deleted the 43-sprint-2.5-documentation-updates branch October 5, 2025 01:48
pipewrk added a commit that referenced this pull request Nov 8, 2025
…dates

docs: Sprint 2.5 - Namespace Architecture Documentation Updates
pipewrk added a commit that referenced this pull request Nov 8, 2025
- Creates 7 issues for remaining Sprint 2.5 tasks (Tasks 2-8)
- Task 1 (Documentation) already completed via PR #44
- Covers namespace architecture implementation (Tasks 2-6)
- Includes Sprint 2 completion work (Tasks 7-8)
- Ready to create milestone and issues for 3-day sprint

Usage: ./scripts/create-sprint-2.5-issues.sh
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