is this sentence gramatically correct?

Obviously your opinion differs not only from mine but also all the punctuation guides I have seen, including the one I quoted. Here are some more:

Those are all from the first page of a Google search for “punctuation dash”. All those that give an opinion on the matter suggest it should be used sparingly. All agree that it marks a strong interruption - “dramatic”, “violent”, “flashy”, “abrupt” - one that (unlike parentheses or bracketing commas) adds emphasis to the material it sets off. The set-off material in this case does not demand emphasis. It is a quiet, confiding aside (as Leo Bloom nicely explained). pulykamell wants to leave it out altogether, but, if he can’t have that, he wants to draw extra attention to it with a dash. That makes no sense at all.

Nutters, the lot of them. The dash is most certainly not a shout. While emphatic, it strikes me as more in the nature of an aside. Were I reading the text aloud, a dash would indicate to me that I should lower the tone of the final clause in comparison to the previous sentence.

It is as far as most people are concerned.

Grammar (or, quite often, ‘grammer’) is “any set of rules, prescriptions, or irrational biases held towards language or how language is used, especially by other people. It is purely prescriptive, by which is meant ‘arbitrary and inconsistent and utterly incompatible with how the language is used by skilled speakers and authors.’” Truly, if you can find a skilled L1 speaker of a given language who follows all of that language’s ‘grammatical’ rules, your knowledge of ‘grammar’ is so deficient you missed the last dozen mistakes that person made.

Of course, he is punctual, conscientious and resourceful. These are all qualities that you would appreciate.

That’s how I would phrase it. A comma before the and in American English is largely considered outdated and is omitted. While both phrases reflect one another, the latter is not an independent phrase which is one reason to use a semicolon. Another reason to use a semicolon is in grouping with commas to show another item within the grouping. Sometimes people say that others use a semicolon to show that they went to college.

You did not just open the serial comma debate, did you?

dives for trenches

Ok, you are right and every punctuation expert is wrong.

This is best.

Unless, he/she can’t stand employees who are on time, respectful of others, or good at pulling things together from little to work with. Or simply for being told what he/she does or doesn’t appreciate.

Then again, that would make him/her an idiot.

I wonder… if this person ever saw the discussion being had about this one sentence, they’d probably dismiss the letter on the whole because of the insane levels of pedantry over such a minor detail that would otherwise go completely unnoticed.

Your words, not mine. prints a run of t-shirts

And for the record, none of the texts I have on hand that discuss punctuation use describe it as meaning a shout and in fact, one of the example sentences is “Health, friends, family — where would we be without them?” which is about as untwisted and demure a sentence as ever existed in the world. So, either one must admit that some experts must be wrong or that the dash can really be used as one likes.

Here is the Straight Dope FAQ on the serial comma. It is a matter of style.

Rather than let a pointless argument die a natural death, I will add that a dash is casual or conversational, and a letter of recommendation is neither.

A semicolon is usually followed by an independent clause (a subject and a verb). I understand what you’re trying to do - to create a pause before the words “all qualities.” You have three choices.

You could use a colon. This would be the most formal, and possibly the most suitable for a resume or letter of recommendation.

You could use a comma. This is also grammatically correct, but not as clear as using a colon, because there are other commas in the sentence.

You could use a dash. This would be the most information option.

While I’m wary of accepting punctuation advice from people who don’t know how to spell “mantelpiece”, it seems to me that the first example here is similar to the sentence under discussion. This example doesn’t seem particularly emphatic; it is merely a list of “people she loved”.

The additional words serve no purpose, and even detract from the message. I would strike them; I prefer trenchant to discursive.

It is; that’s fine; sounds good just the way it is. FWIW: I have been on the receiving end of many, many letters of recommendation over the years. In the interests of full disclosure: the semi-colon is my favorite punctuation mark.