starmen1265/kb-skills
Archived
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix/ci-deploy-reset #20

Merged
starmen1265 merged 37 commits from fix/ci-deploy-reset into main 2026-03-17 00:26:42 +08:00
Conversation 0 Commits 37 Files Changed 80
+5143 -610
starmen1265 commented 2026-03-17 00:26:29 +08:00
Owner
Copy Link
Copy Source
No description provided.
starmen1265 added 37 commits 2026-03-17 00:26:29 +08:00
docs: start milestone v1.1 Architecture Refactor 643acbf165
docs: define milestone v1.1 requirements 90b5ec2a27
docs: create milestone v1.1 roadmap (2 phases) 0bc5bb7584
docs(05): capture phase context 2eff597c3b
docs(state): record phase 5 context session c92ace8e2d
docs(05): create phase 5 plans for archive and directory optimization a11a2dde99
fix(05): revise plan to include KB_INTERMEDIATE_PATH config support ae69878b59 
- Add INT-03 requirement to 05-01-PLAN.md requirements field
- Add Task 1 for KB_INTERMEDIATE_PATH config in KBSettings
- Update Task 2 and Task 3 to use configurable path instead of hardcoded
- Update 05-CONTEXT.md to reflect new decision (config support vs hardcoded)

Addresses checker issue: requirement_coverage for INT-03

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
docs(05): add INT-03 to Phase 5 requirements 18bfb8886d 
- Update ROADMAP.md to include INT-03 (configurable intermediate path)
- Update 05-01 plan description to reflect INT-03 coverage

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
feat(05-01): add KB_INTERMEDIATE_PATH config to KBSettings c5b2c66bc7 
- Add intermediate_path field with default ~/work/intermediate/analysis/
- Follow pydantic-settings naming convention for env var override
- Users can customize via KB_INTERMEDIATE_PATH environment variable

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
feat(05-01): update Pipeline to use configurable intermediate directory 10f4dfa22e 
- Change default output_dir from cwd to settings.intermediate_path
- Reorder settings initialization before output_dir assignment
- Maintain output_dir parameter for testing/custom override

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
feat(05-01): update CLI to use configurable intermediate directory default 7f9caa32d2 
- Import get_settings at top of process_command
- Change default output_dir from cwd to settings.intermediate_path
- Update docstring to document default behavior
- Users can still override with -o flag

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
docs(05-01): complete unified intermediate directory plan 129a6311f4 
- Added intermediate_path config field with KB_INTERMEDIATE_PATH env var support
- Updated Pipeline to use configurable intermediate directory
- Updated CLI to default to intermediate_path instead of cwd
- Completed requirements: INT-01, INT-02, INT-03

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
feat(05-02): add ArchiveInput and ArchiveOutput schemas 9c74f51b43 
- Add ArchiveInput with content_dir, analysis_file, content_type fields
- Add ArchiveOutput with archive_file, skipped, platform fields
- Update PipelineConfig default steps to include archive step
feat(05-02): implement ArchiveProcessor 63dd88279c 
- Create ArchiveProcessor with STEP_NAME = archive
- Implement process() method with idempotent execution
- Add _find_existing_archive() for duplicate detection
- Add _ask_overwrite() using rich.prompt with fallback
- Only copy md files (not json) to archive directory
- Export ArchiveProcessor in __init__.py
feat(05-02): integrate ArchiveProcessor into Pipeline d1497ca550 
- Add ArchiveInput creation in Pipeline.run()
- Import ArchiveInput in pipeline.py
- Add archive_file and skipped to output field recording
- Add ArchiveProcessor to CLI processors list
- Update PipelineConfig steps to include archive
docs(05-02): complete ArchiveProcessor implementation plan bd5ffab6e7
test(05-03): add ArchiveProcessor unit tests be783f97fa 
- Test archive copies to correct platform directory
- Test filename format {YYYYMMDD}_{content_id}_analysis.md
- Test skip when step already completed
- Test detect existing archive by content_id
- Test handle missing analysis file with error

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
test(05-03): add Pipeline archive integration tests 360aee200f 
- Test pipeline includes archive step
- Test archive uses platform directory structure
- Test no date-based subdirectories in archive
- Update default steps assertion to include archive

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
fix(05-03): fix pre-existing test issues blocking verification ccfa201250 
- Fix test_workflow_skips_completed_steps to mock _ask_rerun_step
- Fix test_fetch_video_metadata_success to use AsyncMock
- Document other pre-existing test issues in deferred-items.md

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
docs(05-03): complete archive testing plan ea672f4526 
- Add SUMMARY.md with 8 tests (5 unit + 3 integration)
- Update STATE.md: Phase 5 complete, advance to Phase 6
- Mark requirements OUT-01, OUT-02, OUT-03, OUT-04 complete
- Document test patterns and decisions

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
docs(phase-05): complete phase execution - archive-directory bee61f6e08 
- Phase 5 归档与目录优化完成
- 所有 11 个需求 (INT/ARC/OUT) 已验证通过
- 测试: 8 个归档相关测试通过

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
docs(06): capture phase context 9f4aee0ac7
docs(state): record phase 6 context session 24806adb90
docs(phase-06): research code consolidation domain 33acdf0c2e
docs(06): add research and validation strategy 5fc7000ef7
docs(06): create phase 6 consolidation plans 444ac22e44
feat(06-01): create metadata directory and move bilibili modules e2edce2e29 
- Rename kb-workflow to kb_workflow for Python package compatibility
- Move bilibili_extractor.py and bilibili_metadata.py to kb_workflow/metadata/
- Create metadata/__init__.py with BilibiliExporter export

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
feat(06-01): move zhihu modules to metadata directory a4a7454cd0 
- Move zhihu_extractor.py, zhihu_metadata.py, zhihu_article_crawler.py, markdown_converter.py
- Update metadata/__init__.py to export ZhihuPlaywrightExtractor

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
refactor(06-01): update platform adapters to use standard imports 98f8ec2e27 
- Remove sys.path manipulation from bilibili.py and zhihu.py
- Use standard imports from kb_workflow.metadata
- Remove try/except wrappers around imports

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
refactor(06-01): delete original metadata directories and update tests 766283366e 
- Remove kb-bilibili-metadata/ and kb-zhihu-metadata/ directories
- Create kb_workflow/__init__.py to export main and process_command
- Update test_cli/test_process_command.py imports for new package structure

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
docs(06-01): complete metadata consolidation plan 65fdf4646a
feat(06-02): move pipeline core files to kb_workflow.pipeline 279ae684ea 
- Move pipeline.py, schemas.py, config.py, state.py, errors.py
- Move retry.py, idempotency.py, link_parser.py
- Create pipeline/__init__.py with updated imports
- Update all internal imports to use kb_workflow.pipeline prefix

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
feat(06-02): move processors and platforms to kb_workflow.pipeline fdca690bd6 
- Move all processor implementations (parse, download, asr, analyze, archive)
- Move all platform adapters (base, bilibili, zhihu)
- Update all internal imports to use kb_workflow.pipeline prefix

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
refactor(06-02): update CLI and test imports to kb_workflow.pipeline c9ca1b538d 
- Remove sys.path manipulation from kb_workflow.py
- Update CLI imports to use kb_workflow.pipeline prefix
- Update all test file imports and patch paths

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
chore(06-02): remove original kb_pipeline directory 2df08c6e26 
All code has been moved to kb_workflow.pipeline with updated imports.
278 tests pass (14 pre-existing mock-related failures unrelated to refactoring).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
docs(06-02): complete pipeline module consolidation plan 190a04068b 
- Move kb_pipeline/ to kb_workflow.pipeline/
- Update all imports to use absolute paths
- 278 tests pass (14 pre-existing failures unrelated to refactoring)
- CLI works with new module structure

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
docs(06): add phase verification report
PR Test / lint (pull_request) Successful in 2s
Details
3edbdcc183 
9/9 core functions verified, but gaps found:
- 4 remaining sys.path manipulations
- Pipeline not exported at package level

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
starmen1265 merged commit 7a1b2fc244 into main 2026-03-17 00:26:42 +08:00
This repo is archived. You cannot comment on pull requests.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
starmen1265
No Assignees
1 Participants
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starmen1265/kb-skills#20
Delete Branch "fix/ci-deploy-reset"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?