Skip to content

feat(P7): Add wiki components section with datasets, recipes, libraries, and notebooks#19

Merged
hangtime79 merged 1 commit into
masterfrom
feature/P7-wiki-components-section
Dec 1, 2025
Merged

feat(P7): Add wiki components section with datasets, recipes, libraries, and notebooks#19
hangtime79 merged 1 commit into
masterfrom
feature/P7-wiki-components-section

Conversation

@hangtime79

Copy link
Copy Markdown
Owner

Phase 7: Wiki Components Section

Implements all 4 Phase 7 features for the Discovery Agent Enhancement Plan.

Features Implemented

P7-F001: Generate Datasets Section

  • Added _generate_datasets_section() method
  • Generates collapsible HTML <details> block with dataset metadata
  • Shows: name, type, schema columns, description, tags
  • 6 unit tests

P7-F002: Generate Recipes Section

  • Added _generate_recipes_section() method
  • Generates collapsible HTML <details> block with recipe details
  • Shows: name, type, inputs → outputs, code snippet preview
  • 6 unit tests

P7-F003: Generate Libraries & Notebooks Sections

  • Added _generate_libraries_section() method
  • Added _generate_notebooks_section() method
  • Simple bulleted lists for both sections
  • 7 unit tests

P7-F004: Integrate Components Section

  • Added _generate_components_section() orchestrator method
  • Integrated into generate_wiki_article() method
  • Positioned between navigation menu and description
  • 1 integration test

Technical Details

  • Files Modified: 3

    • dataikuapi/iac/workflows/discovery/catalog_writer.py (+130 lines)
    • dataikuapi/iac/workflows/discovery/tests/test_catalog_writer.py (+59 lines)
    • tests/iac/workflows/discovery/unit/test_catalog_writer_components.py (new file, +372 lines)
  • Test Coverage:

    • 20/20 new Phase 7 tests passing (100%)
    • All existing tests still passing (41/41)
    • Total: 61/61 tests passing (100%)
  • Backward Compatibility:

    • Only generated for EnhancedBlockMetadata instances
    • Basic BlockMetadata instances unaffected
    • Empty detail lists return empty strings (graceful degradation)

Dependencies

Note: This PR includes the P6 navigation menu feature as a dependency, since P7 was built on top of P6. Both features are integrated in the wiki article generation workflow.

  • Phase 6 (P6-F001): Navigation menu - required for proper wiki structure
  • Phase 7 (P7-F001-F004): Components section - this PR's main contribution

Wiki Article Structure

With this PR, EnhancedBlockMetadata wiki articles will have:

  1. Frontmatter
  2. Title
  3. Quick Summary
  4. 🗺️ Navigation Menu ← P6
  5. Internal Components ← P7 (NEW!)
    • Datasets subsection (collapsible)
    • Recipes subsection (collapsible)
    • Libraries subsection
    • Notebooks subsection
  6. Description
  7. Inputs/Outputs
  8. Contains
  9. Dependencies
  10. Usage
  11. Changelog

🤖 Generated with Claude Code

Co-Authored-By: Claude [email protected]

…notebooks)

Implements Phase 7 of Discovery Agent Enhancement Plan:

- P7-F001: Add datasets subsection with collapsible HTML details
- P7-F002: Add recipes subsection with code snippet preview
- P7-F003: Add libraries and notebooks subsections
- P7-F004: Integrate all subsections into wiki article

Components section appears between navigation menu (P6) and description.
Only generated for EnhancedBlockMetadata (backward compatible).

Tests:
- 20 new unit/integration tests (100% pass)
- All existing tests still pass

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
@hangtime79 hangtime79 force-pushed the feature/P7-wiki-components-section branch from d8f3269 to 80a5b68 Compare December 1, 2025 12:03
@hangtime79 hangtime79 merged commit f70ecf6 into master Dec 1, 2025
4 checks passed
@hangtime79 hangtime79 deleted the feature/P7-wiki-components-section branch December 1, 2025 12:04
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.

1 participant