Skip to content

Enhance abstract skill for DITA short descriptions#4814

Merged
Lennonka merged 14 commits into
theforeman:masterfrom
Lennonka:enhance-abstract-skill-for-DITA-shortdesc
May 27, 2026
Merged

Enhance abstract skill for DITA short descriptions#4814
Lennonka merged 14 commits into
theforeman:masterfrom
Lennonka:enhance-abstract-skill-for-DITA-shortdesc

Conversation

@Lennonka

@Lennonka Lennonka commented May 5, 2026

Copy link
Copy Markdown
Contributor

What changes are you introducing?

Enhancing the abstract AI skill with instructions for DITA short descriptions

Why are you introducing these changes? (Explanation, links to references, issues, etc.)

To achieve higher quality of AI work

Anything else to add? (Considerations, potential downsides, alternative solutions you have explored, etc.)

These instructions come from DITA reference.

Contributor checklists

  • I am okay with my commits getting squashed when you merge this PR.
  • I am familiar with the contributing guidelines.

Please cherry-pick my commits into:

  • Foreman 3.18/Katello 4.20 (Satellite 6.19)
  • Foreman 3.17/Katello 4.19
  • Foreman 3.16/Katello 4.18 (Satellite 6.18; orcharhino 7.6, 7.7, and 7.8)
  • Foreman 3.15/Katello 4.17
  • Foreman 3.14/Katello 4.16 (Satellite 6.17; orcharhino 7.4; orcharhino 7.5)
  • Foreman 3.13/Katello 4.15 (EL9 only)
  • Foreman 3.12/Katello 4.14 (Satellite 6.16; orcharhino 7.2 on EL9 only; orcharhino 7.3)
  • We do not accept PRs for Foreman older than 3.12.

@github-actions github-actions Bot added Needs tech review Requires a review from the technical perspective Needs style review Requires a review from docs style/grammar perspective labels May 5, 2026
Comment thread .cursor/skills/abstract/SKILL.md Outdated
@Lennonka Lennonka added Needs testing Requires functional testing and removed Needs tech review Requires a review from the technical perspective labels May 5, 2026
@github-actions

github-actions Bot commented May 5, 2026

Copy link
Copy Markdown

The PR preview for e688592 is available at theforeman-foreman-documentation-preview-pr-4814.surge.sh

No diff compared to the current base

show diff

@Lennonka Lennonka requested a review from aneta-petrova May 5, 2026 20:19
@Lennonka

Lennonka commented May 5, 2026

Copy link
Copy Markdown
Contributor Author

Going to test it.

@Lennonka Lennonka marked this pull request as draft May 5, 2026 20:53
@Lennonka Lennonka mentioned this pull request May 5, 2026
9 tasks
@Lennonka

Lennonka commented May 5, 2026

Copy link
Copy Markdown
Contributor Author

TBH, I have no idea if the results are better or worse now... see #4815 (raw AI output, single latest attempt, no manual editing)

@aneta-petrova @jafiala Care to try it out too?

@Lennonka Lennonka marked this pull request as ready for review May 5, 2026 22:30
Comment thread .cursor/skills/abstract/SKILL.md Outdated
@Lennonka Lennonka requested a review from jafiala May 6, 2026 06:45
@Lennonka

Lennonka commented May 6, 2026

Copy link
Copy Markdown
Contributor Author

This took two attempts and minor manual editing, but I think the result is pretty good: #4822

Comment thread .claude/skills/abstract/SKILL.md
Comment thread .cursor/skills/abstract/SKILL.md Outdated
Comment thread .cursor/skills/abstract/SKILL.md
Comment on lines +30 to +33
- For module-type-specific abstract rules, look up the reference file that matches the `:_mod-docs-content-type:` AsciiDoc attribute in the module (open and apply that file in full):
- `CONCEPT` → [references/concept.md](references/concept.md)
- `PROCEDURE` → [references/procedure.md](references/procedure.md)
- `REFERENCE` → [references/reference.md](references/reference.md)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Would it be possible to include the specific guidelines here instead of in separate files? Or does the approach with separate files have any benefits?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Or actually, how about we pull in modular templates as a separate skill/skill reference, and include these specifics in those templates? Pulling the templates into this repo was proposed before in #4722, and I was against it, but I'm not sure how much longer the modular guides and templates will keep being supported and updates. Perhaps it's time we took ownership of them here as a project resource? Then these abstract instructions could be integrated with the abstract part of each template.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

What do modular templates have to do with the content of short descriptions?

Separating the specific instructions for each type - I did that based on best practices for agent skills.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Fair enough, I can look into it myself later.

Comment thread .cursor/skills/abstract/references/concept.md Outdated

- Introduce the concept and provide a concise answer to the question "What is this?" and in some cases "Why do I care about this?"
- If the concept is unfamiliar, start with a brief definition.
- Avoid using the short description to lead in or build up to a topic.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What does this mean? A short description does introduce a topic.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It's supposed to summarize the topic, not introduce it.

Comment thread .cursor/skills/abstract/references/concept.md Outdated
Comment thread .cursor/skills/abstract/references/concept.md Outdated
Comment thread .cursor/skills/abstract/references/concept.md Outdated
Comment thread .cursor/skills/abstract/references/reference.md Outdated

When the module has `:_mod-docs-content-type: REFERENCE`, follow these rules for the abstract (short description):

- Briefly describe what the reference item does, what it is, or what it is used for.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

"reference item" seems unclear

@Lennonka Lennonka May 20, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It's the "thing being referenced in the module". I don't know how else to call it. It seems clear enough to me. I'm open to suggestions.

Comment thread .cursor/skills/abstract/SKILL.md Outdated
Comment thread .cursor/skills/abstract/references/procedure.md
Comment thread .cursor/skills/abstract/SKILL.md Outdated
@pr-processor pr-processor Bot added Waiting on contributor Requires an action from the author Needs re-review and removed Waiting on contributor Requires an action from the author Needs re-review labels May 20, 2026
@Lennonka Lennonka requested a review from aneta-petrova May 20, 2026 13:49
@jafiala

jafiala commented May 21, 2026

Copy link
Copy Markdown
Contributor

Seems to work well enough for me.
#4871
Only thing, it used "using" instead of "by using" a couple times.

@Lennonka Lennonka force-pushed the enhance-abstract-skill-for-DITA-shortdesc branch from 4550ebe to e688592 Compare May 21, 2026 15:59
@Lennonka

Copy link
Copy Markdown
Contributor Author

Rebased.

@Lennonka

Lennonka commented May 21, 2026

Copy link
Copy Markdown
Contributor Author

Seems to work well enough for me. #4871 Only thing, it used "using" instead of "by using" a couple times.

Thank you, @jafiala! This is one of those details we will have to clean up when we start adding Rules because it isn't specific to any skill.

@Lennonka

Copy link
Copy Markdown
Contributor Author

I've tested it on #4870 and got quite good results too. Marking it as "testing done".

@Lennonka Lennonka added testing done No issues from the functional perspective and removed Needs testing Requires functional testing labels May 21, 2026
@Lennonka

Copy link
Copy Markdown
Contributor Author

Since there haven't been any further comments, I'm considering this acked.

@Lennonka Lennonka merged commit 46c4b23 into theforeman:master May 27, 2026
8 checks passed
Lennonka added a commit that referenced this pull request May 27, 2026
Co-authored-by: Aneta Petrová <apetrova@redhat.com>
Lennonka added a commit that referenced this pull request May 27, 2026
Co-authored-by: Aneta Petrová <apetrova@redhat.com>
@Lennonka

Copy link
Copy Markdown
Contributor Author

Cherry picked:

@Lennonka Lennonka deleted the enhance-abstract-skill-for-DITA-shortdesc branch May 27, 2026 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs style review Requires a review from docs style/grammar perspective testing done No issues from the functional perspective

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants