Skip to content

[8.2] Resolve all javadoc and javac errors and warnings #2200

Description

@vijaya-boddipudi

Description Across the PercussionCMS codebase, multiple modules still report Java compiler (javac) and/or Javadoc errors and warnings during the build. These diagnostics should be cleaned up to improve code quality and maintain a warning-free, documentation-complete build under JDK 21. This issue tracks resolving all remaining javac and javadoc errors and warnings across the repository (not limited to a single module). Module-level cleanups such as #2045 are examples of the work required. Cleaning these issues will improve API documentation consistency and help maintain overall code quality. Expected Result - The full project should build without any javac or javadoc warnings or errors. - All source files should compile cleanly under JDK 21 with the project's -Xlint:all settings. - Public classes, interfaces, methods, constructors, fields, and parameters should contain complete and valid Javadoc. - Javadoc generation should complete cleanly without warnings. Actual Result The project builds with modules that still report javac and/or javadoc diagnostic messages (errors and/or warnings) during compilation and documentation generation. Acceptance Criteria - Resolve all javac and javadoc errors and warnings across all affected modules. - Ensure all public APIs have complete and valid Javadoc. - Verify the project builds successfully with zero javac errors, zero javadoc errors, zero javac warnings, and zero javadoc warnings. - Confirm no regressions are introduced during cleanup. Documents to refer and follow: java-api-writing-specs.md javadoc-checklist.md javadoc-spec-summary.md javadoc-tool-architecture.md javadoc-tool-comments.md

Agent progress (night-issue-prs)

Slice Issue Status PR Notes Updated
perc-deployer ContentRelation Xlint #3017 pr_opened #3046 #2028 child; residual #3047 ContentType 2026-08-11
ContentType handler (deployer) #3047 pr_opened #3059 PSContentTypeDependencyHandler ~69→0 2026-08-11T20:45Z
#3061 this-escape DTO/parser + serial-field #3061 done #3106 this-escape 5→0, serial-field 14→0; residual #3107 2026-08-12
#3107 sitemanage misc Xlint after #3061 #3107 pr_opened #3158 main Xlint 40→0; residual test-source #3159 2026-08-12T07:51Z
sitemanage test-source Xlint after main zero (#3107) #3159 pr_opened #3169 test-source 60→0; path-injection without Unsafe 2026-08-12T10:41Z
utils javac residual after batch 8 (WorkflowUtilsBase/XmlSer/casts) #3172 pr_opened #3347 XmlSer Class<?> + Collection className + ItemIterator asItem; #3277 CRTP merged; WorkflowUtilsBase/inherent casts remain 2026-08-13

Activity

  1. added this to the 8.2 milestone on Aug 6, 2026
  2. 188 remaining items

  3. natechadwick-intsof commented on Aug 12, 2026

    @natechadwick-intsof
    Collaborator

    night-issue-prs (2026-08-12T07:51Z): #3107 main-source misc Xlint PR #3158 (40→0). Residual test-source #3159. Fixes #3107 pending merge.

    Co-Authored by Grok Build using grok-4.5 with agent main.

  4. natechadwick-intsof commented on Aug 12, 2026

    @natechadwick-intsof
    Collaborator

    night-issue-prs: sitemanage #3159 test-source Xlint PR: #3169 (60→0). Parent module #2032.

    Co-Authored by Grok Build using grok-4.5 with agent main.

  5. natechadwick-intsof commented on Aug 12, 2026

    @natechadwick-intsof
    Collaborator

    night-issue-prs: utils batch 8 (#3015) PR #3173 opened; residual #3172 filed for WorkflowUtilsBase raw + XmlSer + cast residuals.

    Co-Authored by Grok Build using grok-4.5 with agent main.

  6. natechadwick-intsof commented on Aug 12, 2026

    @natechadwick-intsof
    Collaborator

    night-issue-prs: #3172 skipped (no code change).

    Triage: intentional utils residuals after batch 8 — PSWorkflowUtilsBase public raw List/Map APIs need explicit source-compat policy; PSXmlSerializationHelper is a larger redesign; remaining casts are scoped helpers. Not a safe unattended strip vs strip-ready batches.

    Progress row on this parent updated to skipped. Issue #3172 left open for a future policy/redesign-driven batch (no residual children filed — inventory already on #3172).

    Co-Authored by Grok Build using grok-4.5 with agent main.

  7. natechadwick-intsof commented on Aug 13, 2026

    @natechadwick-intsof
    Collaborator

    #3172 slice: PR opened #3277 — CRTP self()/cloneConfig typed on ConfigurationContextAbstract. WorkflowUtilsBase / XmlSer / inherent Map-deser casts remain on #3172.

    Co-Authored by Grok Build using grok-4.5 with agent main.

  8. natechadwick-intsof commented on Aug 13, 2026

    @natechadwick-intsof
    Collaborator

    night-issue-prs triage: New javac residual children — DCE #2045 → #3286/#3287/#3288; perc-system #2299 → #3289/#3290/#3291.

    Co-Authored by Grok Build using grok-4.5 with agent night-issue-prs-triage.

  9. natechadwick-intsof commented on Aug 13, 2026

    @natechadwick-intsof
    Collaborator

    night-issue-prs: #3172 second residual slice opened as PR #3347 (XmlSer Class<?> + PSCollection className ctor + PSItemIterator asItem). Prior CRTP slice is merged #3277. WorkflowUtilsBase public raw API and inherent Copier/restoreList casts remain policy/inherent. modules/utils mvnw clean install BUILD SUCCESS, Tests run: 387.

    Co-Authored by Grok Build using grok-4.5 with agent main.

  10. natechadwick-intsof commented on Aug 18, 2026

    @natechadwick-intsof
    Collaborator

    Reconcile: perc-toolkit module tracker #2030 closed (main/test Xlint + javadoc 0 via #2658 #2982 #2990). Umbrella stays open for remaining module trackers (#2016/#2045/#2022/#2028/#2032).

    Co-Authored by Grok Build 1.0.5 using grok-4.6 with agent night-issue-prs-reconcile.

  11. natechadwick-intsof commented on Aug 18, 2026

    @natechadwick-intsof
    Collaborator

    Reconcile: sitemanage module tracker #2032 closed (main + test Xlint residuals via #3158/#3169). Umbrella stays open for remaining modules.

    Co-Authored by Grok Build 1.0.5 using grok-4.6 with agent night-issue-prs-reconcile.

  12. natechadwick-intsof commented on Sep 12, 2026

    @natechadwick-intsof
    Collaborator

    Reconcile: DCE residual #2326 closed (named batches on main). Umbrella stays open for remaining module trackers #2016 / #2045 / #2022 / #2028.

    Co-Authored by Grok Build 1.0.25 using grok-4.6 with agent night-issue-prs-reconcile.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    8.2This issue found in 8.2 releasep8priority 8tech-debttech debt

    Type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions