Skip to content

feat(ontology): expand source-post Voice-of-X taxonomy (ADR 0246) - #713

Merged
seonghobae merged 5 commits into
mainfrom
feat/voice-of-x-complete-taxonomy
Aug 27, 2026
Merged

feat(ontology): expand source-post Voice-of-X taxonomy (ADR 0246)#713
seonghobae merged 5 commits into
mainfrom
feat/voice-of-x-complete-taxonomy

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Summary

Expands the governed source-post Voice-of-X vocabulary from five to twelve
codes under ADR 0246.

  • Adds Supplier, Employee, Business, Regulator, Investor, Society, and Process
    as SKOS concepts and replay-safe lookup values.
  • Keeps source-post voice separate from named-organization relationships; no
    relationship is inferred from a post's voice.
  • Records the terms as product-controlled categories rather than claiming an
    exhaustive standard taxonomy or an exact literature crosswalk.
  • Renumbers the ADR and migration to avoid collisions with concurrent PRs.
  • Existing API lookup-label behavior exposes the new filter values without
    exposing implementation boundaries in customer copy.

Verification

  • pytest ontology, entity-relationship, shapes, publication, documentation,
    and docstring suites: 69 passed, 1 skipped.
  • Migration replay and static SQL contracts: 30 passed.
  • Ruff and diff hygiene passed.

Exact-head hosted checks and independent current-head review remain
authoritative.

@coderabbitai

coderabbitai Bot commented Aug 26, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 51 minutes.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 7915d704-72d4-44c6-b8d5-455c32990cf7

📥 Commits

Reviewing files that changed from the base of the PR and between e357954 and 203d425.

📒 Files selected for processing (9)
  • CHANGELOG.md
  • docs/adr/0246-expanded-voice-of-x-post-taxonomy.md
  • docs/ontology/lineageweave-kg.ttl
  • docs/product-requirements.md
  • docs/product-technical-gap-baseline.md
  • frontend/src/i18n.test.ts
  • frontend/src/i18n.ts
  • migrations/0235_voice_of_x_post_taxonomy.sql
  • tests/test_ontology.py

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

devin-ai-integration[bot]

This comment was marked as resolved.

@seonghobae
seonghobae enabled auto-merge (squash) August 26, 2026 14:08
@seonghobae seonghobae changed the title feat(ontology): govern complete Voice-of-X taxonomy (ADR 0232) feat(ontology): expand source-post Voice-of-X taxonomy (ADR 0246) Aug 26, 2026
devin-ai-integration[bot]

This comment was marked as resolved.

seonghobae pushed a commit that referenced this pull request Aug 26, 2026
seonghobae pushed a commit that referenced this pull request Aug 26, 2026

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Devin Review found 0 new potential issues.

Open in Devin Review

devin-ai-integration[bot]

This comment was marked as resolved.

seonghobae and others added 5 commits August 27, 2026 17:13
Expand voc_type from five to twelve post types (Supplier, Employee,
Business, Regulator, Investor, Society, Process added) grounded in
stakeholder identification theory and quality-engineering practice, and
mirror six new counterparty relationship codes so any post class can
type each named organization. Ontology SKOS concepts/object properties,
idempotent migration 0222, classifier vocabulary + prompt, round-trip
and parser tests stay in sync.
@seonghobae
seonghobae force-pushed the feat/voice-of-x-complete-taxonomy branch from 850494c to 203d425 Compare August 27, 2026 08:16
@seonghobae
seonghobae merged commit 794a0c1 into main Aug 27, 2026
1 of 2 checks passed

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Devin Review found 2 new potential issues.

Devin Review

Comment thread CHANGELOG.md
Comment on lines -11 to -47
- Evidence-bound occupational construct semantics now keep cognitive
abilities, work styles, work activities, affective reactions, performance
behaviors, and FJA worker functions distinct. Record-to-construct links
require a reified evidence span and PROV-O derivation/time; unsupported
DPT-to-psychology crosswalks and local scores remain unavailable (ADR 0248).
- Versioned occupational construct vocabularies and semantic-unit assertions
now persist in normalized tables. Database and application validation require
same-Post verbatim evidence, and authorized Post detail exposes provenance
without internal identifiers or numerical scores (ADR 0249).
- An operator-only O*NET 31.0 catalog synchronizer now imports every official
cognitive-ability, work-style, and work-activity Content Model element with
stable IRIs, descriptions, attribution, and a deterministic source digest;
conflicting release metadata fails closed (ADR 0250).
- Post-content ingestion now traverses bounded official O*NET hierarchy nodes
through contextual-orchestrator's multi-agent conduct workflow and persists
only exact catalog IRIs with verbatim semantic-unit evidence. A digest-bound
run ledger preserves successful empty extraction without inventing a signal
(ADR 0253).
- Authorized Post detail now shows O*NET work evidence with its verbatim source
span, explicit inference status, and official definition action. Complete
empty, processing, and unavailable states stay distinct and localized rather
than collapsing into a blank panel. Historical cutoff reads omit live
assertions and direct the reviewer to the cutoff-known body (ADR 0254).
- Assertion-backed occupational constructs now appear in the authorized
ontology neighborhood with persisted truth status, cutoff-safe availability,
exact Post evidence, and fail-closed conflicting truth. The projection
duplicates no `knowledge_graph_edge` row and creates no person trait or score
(ADR 0255).

- The DOT/FJA Data/People/Things worker-function taxonomy is now published
in the canonical ontology: all 24 worker functions carry the official
Dictionary of Occupational Titles Appendix B definitions verbatim, their
definitional ordinal ranks (ADR 0232). No DOT-to-O*NET or Fleishman
crosswalk is inferred without an authoritative mapping source. A
deterministic application read
model (`lineageweave.worker_function_taxonomy`) exposes fail-closed
lookups; ranks are scale positions and are never used as weights.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changelog drops entries for features still in the tree

This edit deletes the changelog entries for the occupational-construct (ADRs 0248-0255) and DOT/FJA worker-function work, yet worker_function_taxonomy and migrations 0238-0241 remain in the tree. The changelog no longer documents these present, unreleased features.

Prompt for agents
The CHANGELOG 'Added' section in this PR removes the entries describing the occupational-construct features (ADRs 0248, 0249, 0250, 0253, 0254, 0255) and the DOT/FJA worker-function taxonomy (ADR 0232). These entries existed in the merge-base and the underlying features are still present in the head branch (migrations/0238_*.sql through 0241_*.sql, lineageweave/worker_function_taxonomy.py, and docs/adr/0232-worker-function-taxonomy-in-the-published-ontology.md). This appears to be an accidental loss during a rebase where the branch's older Unreleased/Added block overwrote the newer entries. Restore the removed changelog entries alongside the new Voice-of-X entry so the changelog stays synchronized with the features actually present in the tree.
Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

Comment thread tests/test_ontology.py
Comment on lines +49 to +53

# edge_team_affiliation/edge_mention_organization), 0042 (ADR 0207:
# the original five voc_type post-type codes) + 0235 (ADR 0246: the
# seven further Voice-of-X post-type codes), and
# 0220 (ADR 0222: node_project/edge_mention_project).

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

📝 Info: Migration-list comment no longer mentions 0241

The reformatted comment above _ADDITIONAL_LOOKUP_MIGRATION_PATHS stops at 0220 and drops the 0241 description, but the 0241 path is still in the tuple. Comment-only drift; the list itself is correct.

Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

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